Skip to content

Pin the artefact upload and the rest by commit - #23

Merged
PNX89 merged 1 commit into
mainfrom
pin-the-actions-by-commit
Aug 29, 2026
Merged

Pin the artefact upload and the rest by commit#23
PNX89 merged 1 commit into
mainfrom
pin-the-actions-by-commit

Conversation

@PNX89

@PNX89 PNX89 commented Aug 29, 2026

Copy link
Copy Markdown
Owner

A tag is a pointer its owner can move. Every third-party action in this repository referenced one, so a workflow that passed yesterday could run different code today with no commit here, and the version a reader sees would still say what it always said.

Each reference now names the commit the tag pointed at, with the version in a trailing comment.

azure/setup-helm published an annotated tag, which points at a tag object rather than a commit. Pinning that object would have pinned something its owner can still move, so it is dereferenced to the commit underneath.

Nothing about what runs changes today. What changes is that it cannot change tomorrow without a commit here.

A tag is a pointer its owner can move. Every third-party action here referenced one, so a
workflow that passed yesterday could run different code today without a single commit in this
repository, and the version a reader sees would still say what it always said.

Each reference now names the commit the tag pointed at, with the version in a trailing comment
so a human can still read it. azure/setup-helm published an ANNOTATED tag, which points at a tag
object rather than at a commit: pinning that object would have pinned a thing its owner can
still move, so it is dereferenced to the commit underneath.

Nothing about what runs changes today. What changes is that it cannot change tomorrow without a
commit here.
@PNX89
PNX89 merged commit ff1bf26 into main Aug 29, 2026
5 checks passed
@PNX89
PNX89 deleted the pin-the-actions-by-commit branch August 29, 2026 10:50
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