-
Notifications
You must be signed in to change notification settings - Fork 84
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: composite GitHub actions #3119
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3119 +/- ##
==========================================
- Coverage 56.82% 56.76% -0.06%
==========================================
Files 208 208
Lines 29276 29276
==========================================
- Hits 16635 16618 -17
- Misses 10803 10817 +14
- Partials 1838 1841 +3 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
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.
This certainly looks better than before.
This PR is a blessing for when we have to update to a newer Go version! |
Reuse
setup-go
step throughout the pipeline. This way it would be easier to update go version in the future and we ensure consistency. If this approach is mutually agreed to be better, we can apply it to other parts of the pipeline as well.category: refactor
ticket: none