Skip to content

ci: pin release-please-action to its commit SHA - #115

Merged
cbetta merged 1 commit into
mainfrom
cb/fix-release-please-pin
Jul 28, 2026
Merged

ci: pin release-please-action to its commit SHA#115
cbetta merged 1 commit into
mainfrom
cb/fix-release-please-pin

Conversation

@cbetta

@cbetta cbetta commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary

Follow-up to #114. Renovate's dashboard kept proposing chore(deps): update googleapis/release-please-action digest to 45996ed, which surfaced a subtlety in the pin #114 introduced.

googleapis/release-please-action uses an annotated v5 tag. In #114 I resolved it via git/ref/tags/v5, which returns the SHA of the tag object (0dfd8538), not the commit it points to. This repins to the actual v5.0.0 release commit 45996ed — both v5 and v5.0.0 dereference to it.

This was not breaking

To be clear: the merged #114 pin works at runtime. The runner dereferences the tag object automatically, and the release-please job ran green on the #114 merge commit (run). So no production impact.

The reason to fix it anyway:

  1. Convention — every other action in this repo is pinned to a commit SHA; a tag-object SHA is the odd one out, and SHA-pin security scanners (zizmor/ratchet) expect commit SHAs.
  2. Renovate churn — Renovate compares against the commit digest, so it re-adds this entry to the dashboard every cycle until the pin matches.

Verification

🤖 Generated with Claude Code

#114 pinned release-please-action to 0dfd8538, which is the SHA of the
annotated `v5` tag *object*, not the commit it points to. The runner
dereferences it fine (the release job ran green), but it isn't a commit
SHA, so it breaks the SHA-pin convention and Renovate keeps re-proposing
the correction each cycle.

Repin to 45996ed — the actual v5.0.0 release commit (both `v5` and
`v5.0.0` dereference to it) — matching how every other action here is
pinned and clearing the recurring Renovate dashboard entry.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@cbetta
cbetta marked this pull request as ready for review July 28, 2026 09:11
@cbetta
cbetta merged commit e9f786a into main Jul 28, 2026
6 checks passed
@cbetta
cbetta deleted the cb/fix-release-please-pin branch July 28, 2026 09:11
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