-
Notifications
You must be signed in to change notification settings - Fork 13
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
Adjust Github Actions #532
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #532 +/- ##
=========================================
Coverage 37.99% 37.99%
Complexity 1115 1115
=========================================
Files 181 181
Lines 4427 4427
=========================================
Hits 1682 1682
Misses 2745 2745 ☔ View full report in Codecov by Sentry. |
265c432
to
502a724
Compare
502a724
to
29be8d7
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.
Good to go apart from my comment. Feel free to merge anyway, but I'd be curious about the reasoning.
@@ -30,15 +28,18 @@ env: | |||
|
|||
jobs: | |||
unit-tests: | |||
runs-on: ubuntu-latest | |||
runs-on: ubuntu-22.04 |
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.
What's the reason for this change? As long as there's no incompatibilities with ubuntu-latest
I'd prefer to keep it to not have to manually bump it later. But no hard feelings here 🤷
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 prefer following the principle of having fixed versions instead of floating ones. This way, I aim to avoid any surprises in the future. 😄
permissions
in workflows (should be the default)ubuntu-22.04