-
Notifications
You must be signed in to change notification settings - Fork 44
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 Python 3.11 #1362
Support Python 3.11 #1362
Conversation
ee0bba7
to
b13efe5
Compare
Codecov Report
@@ Coverage Diff @@
## master #1362 +/- ##
=======================================
Coverage 91.46% 91.46%
=======================================
Files 72 72
Lines 4369 4369
=======================================
Hits 3996 3996
Misses 373 373
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
b13efe5
to
7e6103e
Compare
You may have to use 3.11-dev instead of 3.11.0-rc.2 |
setup.py
Outdated
@@ -98,7 +97,7 @@ | |||
"Topic :: Database :: Database Engines/Servers", | |||
"Topic :: Database :: Front-Ends", | |||
], | |||
python_requires=">=3.8,<3.11", | |||
python_requires=">=3.8,<3.12", |
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.
python_requires=">=3.8,<3.12", | |
python_requires=">=3.8", |
I think we should actually open this requirement up completely; this upgrade was fairly painless, beyond pinning one particular sub-dep, and as we are a hybrid library/application we don't want to the hold up for other packages!
3e02dcd
to
cfaa2f8
Compare
cfaa2f8
to
96b4fe2
Compare
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.
LGTM
Closes #1361.