-
Notifications
You must be signed in to change notification settings - Fork 17
workflows: update test matrix to use fedora 41 #140
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
workflows: update test matrix to use fedora 41 #140
Conversation
Reminder: this will probably need manual merging as the test names are changing. I'll do it once I have sufficient reviews. I wonder if I should rename the fedora test jobs something like |
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 wonder if I should rename the fedora test jobs something like
fedora-previous
andfedora-current
or something to avoid this (small bit of) churn? Any opinions?
Well, we can't get rid of changes unless we do something as follows:
latest=$(curl -s -L https://fedoraproject.org/releases.json | jq -r '[.[].version | select("^[0-9]+$")] | max')
previous=$(( $latest - 1 ))
And of course I prefer "latest" over "current".
Would you like to go for it with the current PR?
@Mergifyio rebase |
Fedora 41 has been out for some weeks now. Update the test matrix to remove Fedora 39 and use fedora 41. Signed-off-by: John Mulligan <[email protected]>
Signed-off-by: John Mulligan <[email protected]>
✅ Branch has been successfully rebased |
2b88e58
to
82b7249
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.
LGTM
No, I would like to make that change in a different pr. All this slipped off my radar so I would rather fix the immediate issue and then do future proofing later. |
Fedora 41 has been out for some weeks now. Update the test matrix to remove Fedora 39 and use fedora 41.