-
Notifications
You must be signed in to change notification settings - Fork 21
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
Update GH Action Versions used in Workflows #1339
Conversation
2b64918
to
bc9b971
Compare
bc9b971
to
8767ae6
Compare
8767ae6
to
2359490
Compare
2359490
to
808b47d
Compare
808b47d
to
f429962
Compare
f429962
to
6f92070
Compare
6f92070
to
d8ab0cf
Compare
d8ab0cf
to
dd2471c
Compare
dd2471c
to
fa1b46f
Compare
fa1b46f
to
3b76d54
Compare
mikepenz/gradle-dependency-submission has been archived in favor of gradle/actions/setup-gradle
3b76d54
to
a83b1ef
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.
This looks great! I'll just note that we'll need to make these changes on other Aerie repos (UI and Gateway) as well.
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 to me also - chatted with @Mythicaeda yesterday about a couple questions but I'm happy with this! For reference, this is the successful run of the Publish workflow (not run by PR Checks like the others): https://github.com/NASA-AMMOS/aerie/actions/runs/7995540299
One note: as a result of the changes to upload_artifact
noted by @Mythicaeda above, the "Security Scan" workflow now creates two artifacts called "Security Scan - java" and "Security Scan - javascript". When we do a release, the "Security Scan" artifact was attached to the release by the workflows in aerie-release
, so I will add a PR to aerie-release
to look for the two separate scans in future instead of the one combined one.
Description
Github actions are transitioning from Node 16 to Node 20. This PR is to update our workflows to use the node 20 versions of all actions (v4).
There were some breaking changes in the
upload/download artifact
action, that will impact our usage, specifically being unable to upload two artifacts with the same name. This specifically impacts our Security Scan, Publish, and Test workflows.Verification
Successful execution of each workflow with no Node16 warnings will validate this PR:
Documentation
No doc updates needed.