-
Notifications
You must be signed in to change notification settings - Fork 426
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.13 #10821
ci: test with Python 3.13 #10821
Conversation
|
3d2d76b
to
2e8b7ad
Compare
Datadog ReportBranch report: ✅ 0 Failed, 167 Passed, 1301 Skipped, 3m 0.16s Total duration (33m 34.63s time saved) |
comment in docker-compose for local dev update riot locks for 3.13 limit riot envs that do not build with 3.13
This is blocked by MatthieuDartiailh/bytecode#146 |
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.
Just some NB questions, otherwise LGTM.
I think we need to add a 3.13 image in magic mirror before merging this? Looks like the 3.13 download job is failing
Co-authored-by: erikayasuda <[email protected]>
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.
Looks good from profiling
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.
A few nits, otherwise LGTM
Co-authored-by: Gabriele N. Tornetta <[email protected]>
Co-authored-by: Gabriele N. Tornetta <[email protected]>
This change adjusts CI and the library itself to work under Python 3.13. Any tests that failed under 3.13 are skipped on 3.13 for now and will be unskipped in a future change.
Checklist
Reviewer Checklist