Skip to content

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

Merged
chouetz merged 1 commit into
mainfrom
pin-github-actions-to-sha
Sep 8, 2026
Merged

chouetz merged 1 commit into
mainfrom
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
@Ishirui
Ishirui requested a review from BaptisteFoy August 5, 2026 13:56
@datadog-prod-us1-5

This comment has been minimized.

@Ishirui
Ishirui force-pushed the pin-github-actions-to-sha branch from a4c52ce to 6d3be2f Compare August 5, 2026 14:01
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 6d3be2f to a1fb09f Compare September 8, 2026 08:30
@Ishirui

Ishirui commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351 Bot commented Sep 8, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-09-08 08:30:39 UTC ℹ️ Start processing command /merge


2026-09-08 08:30:46 UTC ℹ️ MergeQueue: Pull request is not mergeable yet

It will be processed automatically as soon as GitHub reports it as mergeable. View in MergeQueue UI.

  • Run /code blockers to see what is blocking it.
  • Run /remove to cancel it.

2026-09-08 08:37:52 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@chouetz
chouetz merged commit 65c9d9e into main Sep 8, 2026
88 of 233 checks passed
@chouetz
chouetz deleted the pin-github-actions-to-sha branch September 8, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants