Skip to content

Fix publish workflow permissions - #55

Open
cowboyd wants to merge 1 commit into
mainfrom
fix/publish-workflow-permissions
Open

cowboyd wants to merge 1 commit into
mainfrom
fix/publish-workflow-permissions

Conversation

@cowboyd

@cowboyd cowboyd commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes publish workflow validation on main by granting contents: write, pull-requests: write, and id-token: write to the calling publish job. The reusable workflow requests these permissions, but the caller currently inherits read-only defaults, so GitHub rejects the workflow before it runs.

Type of change

  • Bug fix
  • Feature
  • Refactor (no behavior change)
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • All tests pass (pnpm test)
  • Files are formatted (pnpm format)
  • I have added/updated tests for my changes (if applicable)
  • I have added a changeset

Application tests and the formatter were not run for this workflow-only change. No application tests or changeset are applicable.

AI-generated code disclosure

  • This PR includes AI-generated code

@pkg-pr-new

pkg-pr-new Bot commented Nov 11, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/bombshell-dev/ui/@clack/ui@55

commit: d272394

@cowboyd
cowboyd marked this pull request as ready for review November 11, 2026 04:02
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