Skip to content

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

Merged
justusschock merged 2 commits into
Lightning-AI:masterfrom
RecreationalMath:ci/pin-action-shas
May 23, 2026
Merged

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

Conversation

@RecreationalMath

@RecreationalMath RecreationalMath commented May 21, 2026

Copy link
Copy Markdown
Contributor

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

codecov-commenter commented May 21, 2026

Copy link
Copy Markdown

⚠️ 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.

@justusschock
justusschock merged commit 8f0737c into Lightning-AI:master May 23, 2026
77 checks passed
@mergify mergify Bot added the ready label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants