-
Notifications
You must be signed in to change notification settings - Fork 44
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
build(snap): update snap to core24 #767
Conversation
32b4d5a
to
61bb971
Compare
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.
Thank you!
Is it something that should be included in changelog ?
We're not currently updating the changelog with individual PRs in rockcraft. I want to start doing that next year (maybe) but for now the changelog is a separate task pre-release. |
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.
Do you need to update the github workflows to test on 3.12?
8ff40ec
to
6030895
Compare
This bumps the default Python version on CI to 3.12, which corresponds to the version of the snap itself.
The package is gone in Python 3.12. This affects unused code, which is now gone, and some spread tests. The spread tests are still valid because they use older bases, so just add a pyright exclusion for them.
6030895
to
3f78479
Compare
Fixes #750