-
Notifications
You must be signed in to change notification settings - Fork 377
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
ci: Test with Python 3.12 #409
Conversation
Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #409 +/- ##
=======================================
Coverage 85.12% 85.12%
=======================================
Files 29 29
Lines 3631 3631
=======================================
Hits 3091 3091
Misses 540 540
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Our test suite builds a test extension, which needs `distutils`, which is no longer part of the standard library as of Python 3.12 and needs to be provided by `setuptools` instead. Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
Python 3.12 is not yet ABI stable, so we should not publish wheels yet. Removing the 3.12 wheels right before publishing allows us to leave them in the GitHub artifact bundle, so we can manually download them from CI if we need them for something. Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
c6a7486
to
c6fc6b2
Compare
OK, this should be ready to be reviewed. The way I'm avoiding publishing the 3.12 wheels is hacky, but I don't see a non-hacky solution, and I manually tested that it works by forcing the |
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 the hacks happen :)
No description provided.