-
Notifications
You must be signed in to change notification settings - Fork 221
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: pin to ubuntu-22.04 for now #4942
Conversation
for more information, see https://pre-commit.ci
# TODO: remove this version pinning | ||
# The latest 0.16 supports workspace inheritance, so the check will fail | ||
version: "0.15" |
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.
I think this is needed to pass the test for now.
Will be fixed by #4941
If we do not merge this, all CI for any PRs will fail, so I will merge for now. |
Thanks a lot! I guess we need to add python onto the images. (would be great if we could pin the images + have dependabot upgrade them, but seems that dependabot doesn't do this...) |
Python is included in ubuntu-24.04. Do you think we need to install something like libpython-dev with apt? Details
|
Ah, interesting. Possibly. I guess we'll hear from others fairly soon on what's required... |
See #4941 (comment)
So far, CI in this repository does not seem to support Ubuntu 24.04.