Skip to content

Pin GHA dependencies by hash#283

Merged
lindstro merged 1 commit intollnl:developfrom
williamjallen:pin-gha-dependencies
Dec 15, 2025
Merged

Pin GHA dependencies by hash#283
lindstro merged 1 commit intollnl:developfrom
williamjallen:pin-gha-dependencies

Conversation

@williamjallen
Copy link
Copy Markdown
Contributor

Hash-pinned dependencies are an important requirement enforced by the OpenSSF scorecard tool: https://github.com/ossf/scorecard/blob/main/docs/checks.md#pinned-dependencies

This commit pins all GitHub Actions workflows to a commit hash.

Hash-pinned dependencies are an important requirement enforced by the OpenSSF scorecard tool: https://github.com/ossf/scorecard/blob/main/docs/checks.md#pinned-dependencies

This commit pins all GitHub Actions workflows to a commit hash.
@lindstro
Copy link
Copy Markdown
Member

Thanks for your contribution. If I understand correctly, adding such hashes would preclude automatic PRs when new versions of actions like checkout and python-setup are rolled out. It probably makes sense to uses hashes for "less trusted" workflow dependencies.

@williamjallen
Copy link
Copy Markdown
Contributor Author

@lindstro Dependabot can update hash-pinned dependencies, including the comment. Given that you already have Dependabot configured to run weekly, there shouldn't be any negative effects on your workflow. An extra bonus of pinning everything is that you can add GHA check to verify that everything is pinned, preventing unpinned dependencies from making it into your workflows in the future. That said, if you don't want to pin the actions/* dependencies, I'd be happy to revert those changes and pin only the 3rd-party actions.

@lindstro
Copy link
Copy Markdown
Member

LGTM

@lindstro lindstro merged commit a6e386b into llnl:develop Dec 15, 2025
1 check passed
@williamjallen williamjallen deleted the pin-gha-dependencies branch December 15, 2025 18:55
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