-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Support typing_extensions.Unpack
#12258
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
typing.Unpack and typing_extensions.Unpacktyping_extensions.Unpack [part 3]
|
Uh-oh. Good to know that Unpack is capricious actually. |
cd4e712 to
6d91311
Compare
typing_extensions.Unpack [part 3]typing.Unpack
|
I've restricted the support for Other than that, please have a look -- I cleaned up the branch history to work from the latest master, but I think I've kept every relevant change. A |
picnixz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be good. I may need to have a look at python/cpython#118168 though.
|
AH ! I remember that test failure ! Let me check what happened |
|
Ah yes, actually, in In addition, with Python 3.11, if you use |
The problem is that we 3.11 and 3.12 are not sync'ed for |
The
typing.Unpackconstruction is supported but nottyping_extensions.Unpackbecause it is treated as aTypeVarinstance (this is due to a hack intyping_extensionsthat sets__class__toTypeVar).This is built on top of #12256.