Skip to content

ci: pin mutable action tags in the workflows this fork actually runs#16

Merged
rldyourmnd merged 1 commit into
mainfrom
ci/pin-mutable-action-tags
Jul 24, 2026
Merged

ci: pin mutable action tags in the workflows this fork actually runs#16
rldyourmnd merged 1 commit into
mainfrom
ci/pin-mutable-action-tags

Conversation

@rldyourmnd

Copy link
Copy Markdown

Eight uses: references pointed at mutable tags — CodeQL's actions/unpinned-tag. A tag can be repointed by its owner at any time, so a tag is not a pin.

The risk is concrete here. actions/checkout@v7 resolves today to 3d3c42e5, while the rest of the estate pins 9c091bb (v7.0.0). The tag already moved.

File Refs
branch-cleaner.yml 1
code-quality.yml 6
pr-title.yml 1

No mutable uses: remains in those three files. actionlint clean.

Scope is deliberately narrow: these are the workflows verified to actually run in this fork. The inherited upstream release and nightly workflows are untouched — gh secret list returns zero secrets here and eleven of them require credentials that cannot exist, so pinning code that cannot execute would be churn. Upstream owns that surface.

Eight uses: references in the three workflows that genuinely run here pointed at
mutable tags, which CodeQL reports as actions/unpinned-tag. A tag can be moved
to a different commit by its owner at any time, so a tag pin is not a pin.

That risk is not hypothetical here: actions/checkout@v7 currently resolves to
3d3c42e5, while the rest of the estate pins 9c091bb (v7.0.0). The tag moved.
These now pin the same commit the estate already vetted rather than whatever v7
points at today.

Scope is deliberately limited to branch-cleaner, code-quality and pr-title —
the three files verified to run in this fork. The inherited release and nightly
workflows are left alone: they require secrets this repository does not have
(gh secret list returns zero), so pinning them would be churn on code that
cannot execute. They need a quarantine decision, not a pin.
@rldyourmnd
rldyourmnd force-pushed the ci/pin-mutable-action-tags branch from 10a5544 to 528928a Compare July 24, 2026 20:25
@rldyourmnd
rldyourmnd merged commit bd60ac8 into main Jul 24, 2026
3 checks passed
@rldyourmnd
rldyourmnd deleted the ci/pin-mutable-action-tags branch July 24, 2026 20:27
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