Skip to content

ci: pin actions to commit SHAs and declare GITHUB_TOKEN permissions - #1

Merged
anglinb merged 1 commit into
devfrom
ci/pin-actions-token-permissions
Nov 12, 2026
Merged

anglinb merged 1 commit into
devfrom
ci/pin-actions-token-permissions

Conversation

@anglinb

@anglinb anglinb commented Nov 12, 2026 •

Copy link
Copy Markdown

The superwall org is about to turn on two Actions policies: require actions to be pinned to a full-length commit SHA and read-only default GITHUB_TOKEN permissions. Either policy breaks workflows that aren't ready, so this PR gets this repo ready first.

What changed

  • Every non-local uses: (actions, reusable workflows, composite-action internals) is pinned to the commit SHA its current tag or branch resolves to, with the version kept as a trailing comment (uses: actions/checkout@<sha> # v4). No versions were changed; annotated tags were dereferenced to their commit.
  • Workflows that relied on the org's read/write default now declare permissions: explicitly, with the minimum each job uses today (contents: read unless a job pushes, comments, creates deployments, etc.).

🤖 Generated with Claude Code

The superwall org is turning on "require actions to be pinned to a full-length
commit SHA" and a read-only default GITHUB_TOKEN.

- Pin every non-local `uses:` (actions, reusable workflows, composite action
  internals) to the commit its current tag or branch resolves to, keeping the
  version as a trailing comment. No version changes.
- Add explicit `permissions:` to workflows that relied on the org's read/write
  default, granting only the scopes each job uses today.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@anglinb
anglinb force-pushed the ci/pin-actions-token-permissions branch from a13995f to ac12168 Compare November 12, 2026 06:41
@anglinb
anglinb merged commit 8237c04 into dev Nov 12, 2026
1 check passed
@anglinb
anglinb deleted the ci/pin-actions-token-permissions branch November 12, 2026 07:01
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