chore: upgrade actions to Node 24 runtime (SHA-pinned) - #62
chore: upgrade actions to Node 24 runtime (SHA-pinned)#62John C. Bland II (johncblandii) wants to merge 4 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Updates GitHub Actions workflow dependencies to SHA-pinned revisions that run on the Node 24 action runtime, reducing Node 20 deprecation warnings and improving supply-chain integrity via immutable pins.
Changes:
- Replaced floating
uses: ...@v*references with commit-SHA pins plus tag-version comments foractions/checkout,actions/setup-node,docker/metadata-action, andaws-actions/configure-aws-credentials. - Applied the same
actions/checkoutSHA pin to the CODEOWNERS validation workflow.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/validate-codeowners.yml | Pins actions/checkout to a specific commit SHA with an upstream tag comment. |
| .github/workflows/lambda.yml | Pins multiple actions (setup-node, checkout, metadata-action, configure-aws-credentials) to specific SHAs with tag comments. |
Suppressed comments (1)
.github/workflows/lambda.yml:105
role-session-namereferences${{ env.IAM_ROLE_SESSION_NAME }}, but the workflow-levelenv:definesIAM_ROLE_SEESION_NAME(typo). As written, this expression will resolve to an empty string and can cause STS to use a default session name, making audit trails and debugging harder. Align the reference (or rename the env key) so the intended session name is actually used.
uses: aws-actions/configure-aws-credentials@e6de054238d6b7531b4efff3b6587d9aade6a06c # v6.2.3
with:
aws-region: us-east-2
role-to-assume: ${{ env.IAM_ROLE_ARN_S3 }}
role-session-name: ${{ env.IAM_ROLE_SESSION_NAME }}
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Also fixed the pre-existing |
|
Same follow-up as ses-lambda-forwarder: the un-typo'd session name |
|
CI triage summary for |
CI triage for this PRbuild-test-distribute — pre-existing, needs a maintainer. The failing test is validate-codeowners — pre-existing, fixed in c39ef08. The |
what
SHA-pinned with precise version comments:
actions/setup-node@v4→@82076278...# v7.0.0actions/checkout@v4→@3d3c42e5...# v7.0.1docker/metadata-action@v5→@dc802804...# v6.2.0aws-actions/configure-aws-credentials@v4→@e6de0542...# v6.2.3why
are already being force-migrated to Node 24
matching the org's direction in chore: upgrade actions to Node 24 runtime and enforce SHA pinning .github#261
references
still on Node 20
mszostok/codeowners-validator@v0.5.0— Docker-based action, not affected by the Node runtime deprecation; no Node 24 release existscloudposse/github-action-auto-release@v1— composite action not in the upgrade matrix; left as-is