Skip to content

ci: pin all GitHub Actions to commit SHAs - #83

Merged
adithya-s-k merged 2 commits into
mainfrom
ci/pin-actions-to-sha
Sep 15, 2026
Merged

adithya-s-k merged 2 commits into
mainfrom
ci/pin-actions-to-sha

Conversation

@adithya-s-k

@adithya-s-k adithya-s-k commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Pin every external action in CI, documentation deployment and release workflows to a verified full commit SHA. This includes the updated action versions from #117 and the Tasksmith jobs added since this PR was opened. Keep version comments so Dependabot can maintain the pins.

Disable uv dependency caches throughout the release workflow, including manual release runs. This preserves setup-uv v10's protection for publishing jobs instead of overriding it with enable-cache: true.

Test plan

  • All eight CI checks passed on b07a52f: Python 3.12/3.13/3.14, recipe and coding-runtime contracts, lint, documentation, and distribution build.
  • actionlint passed.
  • Resolved every SHA through the upstream action repository and reviewed the changes against current main.

Out of scope

No package version change, release publication or dataset changes.

Tags are mutable; a maintainer (or an attacker with maintainer access)
can retag an existing version to point at new code without the
consumer's workflow file changing. Pinning to a full 40-char commit
SHA makes the referenced code immutable while a trailing "# vX.Y.Z"
comment keeps the human-readable version visible and lets Dependabot
(already configured for github-actions in .github/dependabot.yml)
open PRs updating both the SHA and the version comment together when
a new release ships.

Covers all 19 action use-sites across ci.yml, deploy-docs.yml, and
release.yml. Verified every SHA against the action repo's own tag
ref before committing.

Refs: https://docs.github.com/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions#using-third-party-actions
@adithya-s-k
adithya-s-k merged commit 194991e into main Sep 15, 2026
8 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.

1 participant