Skip to content

[ACIX-1926] chore(ci): pin GitHub Actions to commit SHAs - #173

Merged
chouetz merged 1 commit into
masterfrom
pin-github-actions-to-sha
Aug 19, 2026
Merged

[ACIX-1926] chore(ci): pin GitHub Actions to commit SHAs#173
chouetz merged 1 commit into
masterfrom
pin-github-actions-to-sha

Conversation

@Ishirui

@Ishirui Ishirui commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Mechanical rewrite of every uses: reference to a full-length commit SHA, produced by pinact. An unpinned reference resolves to a mutable ref, so a compromise of the upstream action becomes code execution in this repository's CI.

The trailing # vX.Y.Z comment is what lets Renovate and Dependabot keep these bumped, so please keep it.

One thing to check: references that tracked @main or @master were resolved to the latest stable tag. If any of them floated deliberately and that behaviour was load-bearing here, say so on the PR and we will revert that line.

@Ishirui
Ishirui requested a review from a team as a code owner August 5, 2026 13:56
Mechanical rewrite of every `uses:` reference to a full-length commit SHA, produced by [pinact](https://github.com/suzuki-shunsuke/pinact). An unpinned reference resolves to a mutable ref, so a compromise of the upstream action becomes code execution in this repository's CI.

The trailing `# vX.Y.Z` comment is what lets Renovate and Dependabot keep these bumped, so please keep it.

**One thing to check:** references that tracked `@main` or `@master` were resolved to the latest stable tag. If any of them floated deliberately and that behaviour was load-bearing here, say so on the PR and we will revert that line.
@Ishirui
Ishirui force-pushed the pin-github-actions-to-sha branch from e77fecd to a05ab33 Compare August 5, 2026 14:00
@chouetz
chouetz merged commit 33cc248 into master Aug 19, 2026
1 check passed
@chouetz
chouetz deleted the pin-github-actions-to-sha branch August 19, 2026 09:49
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