Skip to content

chore: Pin third-party GitHub Actions to commit SHAs#6567

Open
erezrokah wants to merge 1 commit into
cloudflare:mainfrom
erezrokah:claude/pin-third-party-actions
Open

chore: Pin third-party GitHub Actions to commit SHAs#6567
erezrokah wants to merge 1 commit into
cloudflare:mainfrom
erezrokah:claude/pin-third-party-actions

Conversation

@erezrokah

@erezrokah erezrokah commented Apr 12, 2026

Copy link
Copy Markdown

Summary

Pin all third-party (non-actions/*, non-Cloudflare-owned) GitHub Actions to full 40-character commit SHAs to mitigate supply chain attacks via compromised tags.

Changes

Action Version Files
codecov/codecov-action v5.5.4 _bazel.yml
contributor-assistant/github-action v2.6.1 cla.yml
CodSpeedHQ/action v4.13.1 codspeed.yml
peter-evans/create-pull-request v8.1.1 deps-updater.yml
13rac1/block-fixup-merge-action v2.0.0 fixup.yml
mukunku/tag-exists-action v1.7.0 release.yml
ncipollo/release-action v1.21.0 release.yml
pnpm/action-setup v4.3.0 release.yml, test.yml

Test plan

  • Verify CI workflows pass with SHA-pinned actions

Pin all third-party (non-actions/*, non-Cloudflare-owned) action
references to full 40-character commit SHAs to prevent supply chain
attacks via compromised tags.
@erezrokah erezrokah marked this pull request as ready for review April 12, 2026 13:01
@erezrokah erezrokah requested review from a team as code owners April 12, 2026 13:01
@erezrokah erezrokah requested review from NuroDev and Copilot April 12, 2026 13:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Pins third-party GitHub Actions used in CI/release workflows to full commit SHAs to reduce supply-chain risk from compromised/moved tags.

Changes:

  • Replace third-party uses: <owner>/<repo>@<tag> references with 40-character commit SHAs across workflows.
  • Add inline comments preserving the original action version (e.g., # v4.3.0) for traceability.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/test.yml Pin pnpm/action-setup to a full commit SHA in the workers-sdk test job.
.github/workflows/release.yml Pin third-party actions used for tag checks, releases, and pnpm setup to full commit SHAs.
.github/workflows/fixup.yml Pin 13rac1/block-fixup-merge-action to a full commit SHA.
.github/workflows/deps-updater.yml Pin peter-evans/create-pull-request to a full commit SHA.
.github/workflows/codspeed.yml Pin CodSpeedHQ/action to a full commit SHA.
.github/workflows/cla.yml Pin contributor-assistant/github-action to a full commit SHA.
.github/workflows/_bazel.yml Pin codecov/codecov-action to a full commit SHA.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codspeed-hq

codspeed-hq Bot commented Apr 15, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 14.24%

⚡ 1 improved benchmark
✅ 69 untouched benchmarks
⏩ 129 skipped benchmarks1

Performance Changes

Benchmark BASE HEAD Efficiency
Encode_ASCII_256[TextEncoder][0/0/256] 3.1 ms 2.7 ms +14.24%

Comparing erezrokah:claude/pin-third-party-actions (995979b) with main (c649a48)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@fhanau fhanau self-requested a review April 28, 2026 20:41
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