Skip to content

ci: pin GitHub Actions to commit SHAs with version comments#68919

Merged
dwoz merged 1 commit intosaltstack:3006.xfrom
twangboy:fix/ci/hardpin
Apr 10, 2026
Merged

ci: pin GitHub Actions to commit SHAs with version comments#68919
dwoz merged 1 commit intosaltstack:3006.xfrom
twangboy:fix/ci/hardpin

Conversation

@twangboy
Copy link
Copy Markdown
Contributor

What does this PR do?

Pin all workflow and composite-action uses: references to immutable full commit SHAs instead of floating tags, and add trailing comments (e.g. # v4) so the intended release remains obvious and tooling can update SHAs cleanly. Covers third-party actions (e.g. paths-filter, workflow-conclusion, check-user-permission, action-download-artifact, backport, SSM code signing), official actions/* (checkout, setup-python, upload/download-artifact, upload-artifact/merge, create-release, upload-release-asset, cache), and the composite cache path (aws-actions/configure-aws-credentials, runs-on/cache).

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes/No

@twangboy twangboy added this to the Sulpher v3006.24 milestone Apr 10, 2026
@twangboy twangboy self-assigned this Apr 10, 2026
@twangboy twangboy added the test:full Run the full test suite label Apr 10, 2026
@twangboy twangboy requested a review from a team as a code owner April 10, 2026 00:58
Pin all workflow and composite-action `uses:` references to immutable full
commit SHAs instead of floating tags, and add trailing comments (e.g. # v4)
so the intended release remains obvious and tooling can update SHAs cleanly.
Covers third-party actions (e.g. paths-filter, workflow-conclusion,
check-user-permission, action-download-artifact, backport, SSM code signing),
official `actions/*` (checkout, setup-python, upload/download-artifact,
upload-artifact/merge, create-release, upload-release-asset, cache), and the
composite cache path (`aws-actions/configure-aws-credentials`, `runs-on/cache`).
@dwoz dwoz merged commit d95192b into saltstack:3006.x Apr 10, 2026
415 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants