-
Notifications
You must be signed in to change notification settings - Fork 776
ci/release: Ubuntu 20.04 being sunset #3218
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
Conversation
.github/workflows/release.yml
Outdated
| # Run 'cargo dist plan' (or host) to determine what tasks we need to do | ||
| plan: | ||
| runs-on: "ubuntu-20.04" | ||
| runs-on: "ubuntu-latest" |
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.
ubuntu-latest is often a bad idea - you're kinda begging for CI to fail due to something that isn't your fault. D:
Could you please consider using 22.04 or 24.04 instead?
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.
Yeah no problem, in this specific case using latest would've not broken though. As old versions will eventually be decommissioned.
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.
That's true! Still, I think a PR every few years to update the version is just fine.
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.
anyway, fixed!
1c08ddd to
da2a72d
Compare
Replace 20.04 with latest
da2a72d to
a346aff
Compare
onkoe
left a comment
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! I'm not a maintainer by the way - this PR just kinda caught my eye.
hope it gets merged soon! ;D
emilio
left a comment
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.
lgtm, thanks for doing this!
Replace 20.04 with latest