-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Relax the Python version check for __class_getitem__
tests
#7049
Conversation
`__class_getitem__` to support generics was introduced in Python 3.7. In 3.9 some built-ins were made generic but the functionality for user-defined classes has been there since 3.7.
Classes can be made "generic" using |
yes i know all that. confirming this change ideally should have been made when we in fact added also i thought i just added this to gerrit and it's not there, OK |
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.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision 7bd16d4 of this pull request into gerrit so we can run tests and reviews and stuff
New Gerrit review created for change 7bd16d4: https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3089 |
i wonder why we pinned c03b6e0 at 3.9 |
Well, I haven't got any other insight 😛 |
Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3089 has been merged. Congratulations! :) |
Description
__class_getitem__
to support generics was introduced in Python 3.7.In 3.9 some built-ins were made generic but the functionality
for user-defined classes has been there since 3.7.
Checklist
This pull request is:
must include a complete example of the issue. one line code fixes without an
issue and demonstration will not be accepted.
Fixes: #<issue number>
in the commit messageinclude a complete example of how the feature would look.
Fixes: #<issue number>
in the commit messageHave a nice day!