Skip to content

ci: pin third-party github actions to commit SHAs#136

Merged
zfarrell merged 1 commit into
mainfrom
security/pin-github-actions
Jun 5, 2026
Merged

ci: pin third-party github actions to commit SHAs#136
zfarrell merged 1 commit into
mainfrom
security/pin-github-actions

Conversation

@zfarrell

@zfarrell zfarrell commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Pins all third-party GitHub Actions in ci.yml and release.yml to full commit SHAs to address the supply-chain risk in #46. Reuses the repo's existing v6/v7 pins for consistency.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Automated review unavailable (Claude step failed). Please review manually.

@sentry

sentry Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Automated review unavailable (Claude step failed). Please review manually.

2 similar comments
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Automated review unavailable (Claude step failed). Please review manually.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Automated review unavailable (Claude step failed). Please review manually.

@zfarrell zfarrell closed this Jun 5, 2026
@zfarrell zfarrell reopened this Jun 5, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean, well-scoped supply-chain hardening. All third-party actions in ci.yml and release.yml are SHA-pinned with version comments preserved, and the introduced SHAs are internally consistent with pins already in the repo's merged workflows.

Note: I could not externally verify each SHA against its upstream tag — gh api, git ls-remote, and network fetches are all blocked in this review environment. The reused actions/checkout and actions/download-artifact SHAs already run in merged workflows; the only net-new pin is actions/upload-artifact@b7c566a (# v6), worth a quick manual confirm before merge.

@zfarrell zfarrell merged commit 2494f84 into main Jun 5, 2026
23 checks passed
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.

1 participant