-
Notifications
You must be signed in to change notification settings - Fork 639
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
Bump actions/checkout to v4 for GHA #7284
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## main #7284 +/- ##
==========================================
- Coverage 93.22% 93.21% -0.02%
==========================================
Files 275 275
Lines 59528 59572 +44
Branches 0 7422 +7422
==========================================
+ Hits 55495 55529 +34
- Misses 4033 4043 +10 |
e436d61
to
5c86cd2
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.
PR should have decent description instead of the template. A simple sentence explaining why we need this change is enough in this case. You can also remove the DESCRIPTION line completely though, since that is used to generate the changelog for user facing changes.
Also left one comment. Other than that this looks fine.
@@ -39,10 +39,10 @@ jobs: | |||
needs: params | |||
runs-on: ubuntu-20.04 | |||
container: | |||
image: ${{ needs.params.outputs.build_image_name }}: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.
Why is this needed? It seem unrelated. If it is needed, then that's something that would be helpful to explain in the PR description.
DESCRIPTION: PR description that will go into the change log, up to 78 characters