Skip to content

ci(build): Add packages:write permission to snapshot workflow#3224

Merged
runningcode merged 1 commit intomasterfrom
no/fix-snapshot-permissions
Mar 20, 2026
Merged

ci(build): Add packages:write permission to snapshot workflow#3224
runningcode merged 1 commit intomasterfrom
no/fix-snapshot-permissions

Conversation

@runningcode
Copy link
Contributor

@runningcode runningcode commented Mar 19, 2026

Summary

Fixes failed action here: https://github.com/getsentry/sentry-cli/actions/runs/23303634015

  • Adds packages: write to the snapshot workflow's top-level permissions. GitHub validates reusable workflow permissions statically — even though the docker jobs in build.yml have if: !inputs.is-snapshot and won't run, the caller must still grant the permissions they declare.

🤖 Generated with Claude Code

GitHub validates reusable workflow permissions statically. Even though
the docker jobs in build.yml have `if: !inputs.is-snapshot` and won't
run for snapshots, the caller must still grant the permissions they
declare. Without this, the workflow fails validation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@runningcode runningcode requested review from a team and szokeasaurusrex as code owners March 19, 2026 15:57
@runningcode runningcode merged commit 04366e0 into master Mar 20, 2026
26 checks passed
@runningcode runningcode deleted the no/fix-snapshot-permissions branch March 20, 2026 07:56
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