-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Support typing_extensions.Unpack
#12258
Conversation
typing.Unpack
and typing_extensions.Unpack
typing_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 |
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.Unpack
construction is supported but nottyping_extensions.Unpack
because it is treated as aTypeVar
instance (this is due to a hack intyping_extensions
that sets__class__
toTypeVar
).This is built on top of #12256.