Skip to content

Pin test workflow actions to SHAs - #65

Merged
juan-fernandez merged 3 commits into
mainfrom
juan-fernandez/pin-test-actions-to-shas
Jul 20, 2026
Merged

Pin test workflow actions to SHAs#65
juan-fernandez merged 3 commits into
mainfrom
juan-fernandez/pin-test-actions-to-shas

Conversation

@juan-fernandez

@juan-fernandez juan-fernandez commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Upgrade all five actions/checkout uses in .github/workflows/test.yaml to v7.0.0.
  • Pin them to the immutable commit 9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0.
  • Keep # v7.0.0 comments beside the SHAs for readability and dependency update tooling.

Why

Moving major tags can change after review, while the existing v4 release uses the deprecated Node.js 20 action runtime. Using the latest release at an immutable commit keeps the workflow current and ensures it runs the exact revision that was reviewed.

User impact

No intended product behavior change. The test workflow now uses the Node.js 24-based actions/checkout v7 release.

Verification

  • git diff --check
  • Parsed .github/workflows/test.yaml with Ruby YAML
  • Verified all five checkout references use the v7.0.0 commit SHA and no moving checkout references remain
  • Verified every PR commit signature with git verify-commit

actionlint was not run because it is not installed locally.

@juan-fernandez juan-fernandez added the semver-patch Requests a patch action release after merge label Jul 20, 2026 — with ChatGPT Codex Connector
@juan-fernandez
juan-fernandez marked this pull request as ready for review July 20, 2026 11:05
@juan-fernandez
juan-fernandez requested review from a team as code owners July 20, 2026 11:05
@juan-fernandez
juan-fernandez merged commit 30b363c into main Jul 20, 2026
10 checks passed
@juan-fernandez
juan-fernandez deleted the juan-fernandez/pin-test-actions-to-shas branch July 20, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-patch Requests a patch action release after merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants