Skip to content

Declare contents: write on the publish workflows - #8

Merged
woksin merged 1 commit into
mainfrom
fix/add-release-permissions
Jul 24, 2026
Merged

Declare contents: write on the publish workflows#8
woksin merged 1 commit into
mainfrom
fix/add-release-permissions

Conversation

@woksin

@woksin woksin commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Declare contents: write on the three publish workflows so creating the GitHub release keeps working regardless of the default workflow-token permission.

The three publish workflows create a GitHub release via cratis/release-action
but declared no permissions, relying on the org's default token being writable.
Grant contents: write explicitly so release creation does not depend on the
default and cannot 403 if it is ever tightened to read-only.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@woksin

woksin commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Reviewer context (no user-facing / image change): these three publish workflows had no permissions: block and relied on the org's currently-permissive default GITHUB_TOKEN. cratis/release-action creates the GitHub release, which needs contents: write; every other Cratis release workflow declares it explicitly. This brings them in line so release creation is independent of the org default and won't 403 if the default is tightened to read-only. Only .github/** changes, which are outside the workflows' paths filter, so merging this cuts no release.

@woksin
woksin merged commit 79e47eb into main Jul 24, 2026
@woksin
woksin deleted the fix/add-release-permissions branch July 24, 2026 15:31
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