Skip to content

chore(ci): pin actions to immutable commit SHAs - #14

Merged
yordis merged 1 commit into
mainfrom
yordis/chore-pin-actions
Sep 2, 2026
Merged

chore(ci): pin actions to immutable commit SHAs#14
yordis merged 1 commit into
mainfrom
yordis/chore-pin-actions

Conversation

@yordis

@yordis yordis commented Sep 2, 2026

Copy link
Copy Markdown
Member
  • Mutable tags let an upstream maintainer change what runs here after the fact, and pinning is a prerequisite for turning on the organization SHA pinning policy without taking CI down.

Mutable tags let an upstream maintainer change what runs in CI after review.

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@cursor

cursor Bot commented Sep 2, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CI-only workflow edits that pin existing action versions without changing job logic or application code.

Overview
Pins all third-party GitHub Actions in CI and Release Please workflows to full commit SHAs instead of mutable version tags (@v5.0.0, etc.), with the original tag kept in a trailing comment for readability.

Affected actions: actions/checkout, bufbuild/buf-setup-action, actions/setup-go, golangci/golangci-lint-action, and googleapis/release-please-action. Runtime behavior is unchanged at the pinned revisions; the goal is supply-chain hardening and compatibility with org policies that require SHA pinning.

Reviewed by Cursor Bugbot for commit e7a03f0. Bugbot is set up for automated code reviews on this repo. Configure here.

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 1e4be10c-aaaf-4d5d-a6da-033d9b5e24f2

📥 Commits

Reviewing files that changed from the base of the PR and between 5819153 and e7a03f0.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • .github/workflows/release-please.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The CI and release workflows now pin GitHub Actions to immutable commit SHAs. Existing version comments, Go configuration, and workflow behavior remain unchanged.

Changes

Workflow action pinning

Layer / File(s) Summary
Pin workflow action references
.github/workflows/ci.yml, .github/workflows/release-please.yml
Checkout, Buf setup, Go setup, golangci-lint, and Release Please actions now use immutable commit SHAs. Existing version annotations and Go configuration remain unchanged.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to e7a03

This change makes CI and release automation use immutable action revisions without altering the workflows’ intended behavior. No actionable merge-blocking risk remains beyond normal checks and review.

Poem

A rabbit checks each workflow line
And pins each action firm and fine
SHA by SHA, the paths stay clear
Version notes remain near
CI hops safely through the gate

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: pinning CI actions to immutable commit SHAs.
Description check ✅ Passed The description explains why the CI action references use immutable commit SHAs and relates directly to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yordis/chore-pin-actions

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@yordis
yordis merged commit c9be7a6 into main Sep 2, 2026
5 checks passed
@yordis
yordis deleted the yordis/chore-pin-actions branch September 2, 2026 19:28
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