-
Notifications
You must be signed in to change notification settings - Fork 103
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
Added Python 3.12 to test matrix and classifiers #174
Conversation
You may want to consider updating cibuildwheel as well - I think at least v2.16.2 is needed to build cp312 wheels. IMHO it's a good idea to include this with your PR since you also declare Python 3.12 support, so it'd a bit strange if it'd still do a release without the proper wheels. |
Makes indeed no sense @ThiefMaster, thanks for pointing out! I updated the pull-request. |
.github/workflows/integration.yaml
Outdated
@@ -25,7 +25,7 @@ jobs: | |||
strategy: | |||
max-parallel: 15 | |||
matrix: | |||
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11', 'pypy-3.7', 'pypy-3.8'] | |||
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11', '3.12', 'pypy-3.7', 'pypy-3.8'] |
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.
Let's add pypy-3.9 while here, WDYT?
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.
Feel free to commit, I have no objection.
Just the one comment @foarsitter. I'm happy to push to it, if you prefer. |
Thanks @chayim! |
No description provided.