Skip to content
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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Support Python 3.13 beta #4253

wants to merge 2 commits into from

Conversation

harshil21
Copy link
Member

@harshil21 harshil21 commented May 11, 2024

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

Copy link

@github-actions github-actions bot left a 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 :)

@harshil21
Copy link
Member Author

If possible we should do #4129 before this is merged so it's easier to change setup.py. test_meta fails on 3.13 because of the change to locals() - https://docs.python.org/3.13/library/functions.html#locals. I don't want to change that part yet because I don't want to potentially break the release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant