-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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.13 beta #4253
base: master
Are you sure you want to change the base?
Support Python 3.13 beta #4253
Conversation
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.
Hey! Looks like you edited the (optional) requirements or the pre-commit hooks. I'm just a friendly reminder to keep the additional dependencies for the hooks in sync with the requirements :)
If possible we should do #4129 before this is merged so it's easier to change |
ironically if we don't merge this, then all ptb installations on py 3.13 will fail by default. And we don't have python version specific wheels so we can build it on another version and it should be good to go. uses pre-release of cffi 1.17.0 since they don't plan to release the full version until 3.13 rc1 |
We should probably wait for
cffi
to release their new version to target 3.13, otherwise passport related functionality won't work. Tracking: python-cffi/cffi#23, python-cffi/cffi#71