Skip to content

ci: pin actions by sha#213

Merged
thaJeztah merged 1 commit into
moby:mainfrom
thaJeztah:pin_actions
May 27, 2026
Merged

ci: pin actions by sha#213
thaJeztah merged 1 commit into
moby:mainfrom
thaJeztah:pin_actions

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

No description provided.

@kolyshkin
Copy link
Copy Markdown
Collaborator

To me this (using sha or defining a complete specific version) feels like a totally unnecessary complication which will create extra noise (more frequent updates, etc.).

Unless, of course,

  • a specific action used tends to break often;
  • or GHA is used for something very important and security-sensitive, like building an official release binary.

Neither seems to be the case here.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@kolyshkin
Copy link
Copy Markdown
Collaborator

OTOH looks like everyone else is doing it these days, so why shouldn't we.

@thaJeztah
Copy link
Copy Markdown
Member Author

Yeah, I hate it; I think this should've been something that would be solved through something similar to Go's checksumdb and/or an option to point to a custom DB (with an allow/deny-list of versions / checksums).

I'll bring this one in.

@thaJeztah thaJeztah merged commit 8b533b2 into moby:main May 27, 2026
20 checks passed
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.

2 participants