Skip to content

fix(ci): pin GitHub Actions to commit SHAs#3392

Open
RecreationalMath wants to merge 2 commits into
Lightning-AI:masterfrom
RecreationalMath:ci/pin-action-shas
Open

fix(ci): pin GitHub Actions to commit SHAs#3392
RecreationalMath wants to merge 2 commits into
Lightning-AI:masterfrom
RecreationalMath:ci/pin-action-shas

Conversation

@RecreationalMath
Copy link
Copy Markdown
Contributor

@RecreationalMath RecreationalMath commented May 21, 2026

What does this PR do?

Pins all GitHub Actions and reusable-workflow references to immutable commit SHAs - as agreed with @justusschock in this comment on #3365.

This covers 41 references across 13 files (11 workflows + 2 composite actions). Each action is pinned to the commit its current tag resolves to, with the release version kept as a trailing comment (e.g. actions/checkout@de0fac2… # v6.0.2), the format Dependabot recognises, so it keeps the SHAs current going forward. No behaviour change as every pin points at the same commit the tag already ran.

A few notes for the reviewer:

Before submitting
PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Oh yes, preventing a tj-actions/changed-files style supply chain attack sure is fun.

Pin all third-party and reusable-workflow action references across the
workflows and composite actions to immutable commit SHAs,
keeping the release version as a trailing comment.
No behaviour change, since each action is pinned to the commit its tag
currently resolves to.

greetings.yml is pinned separately in a previous PR.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 21, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37%. Comparing base (d184220) to head (f08eed6).
⚠️ Report is 2 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #3392   +/-   ##
======================================
  Coverage      37%     37%           
======================================
  Files         349     349           
  Lines       19901   19901           
======================================
  Hits         7264    7264           
  Misses      12637   12637           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants