Skip to content

fix: use BUMPY_GH_TOKEN for draft release operations#87

Merged
theoephraim merged 4 commits into
mainfrom
fix/disable-staged-publishing
May 27, 2026
Merged

fix: use BUMPY_GH_TOKEN for draft release operations#87
theoephraim merged 4 commits into
mainfrom
fix/disable-staged-publishing

Conversation

@theoephraim
Copy link
Copy Markdown
Member

@theoephraim theoephraim commented May 27, 2026

Summary

  • Fix draft release functions (createDraftRelease, updateReleaseBody, finalizeRelease, deleteRelease) to use withReleaseToken — they were calling gh directly with GITHUB_TOKEN, which doesn't trigger downstream workflows like on-release.yaml
  • Disable npm staged publishing (not yet ready) and remove the npm install -g npm@latest step from the release workflow

Test plan

  • Verify on-release.yaml triggers when a release is finalized
  • Confirm packages publish normally without staged publishing

Staged publishing is not yet ready for production use. Removes npmStaged
config and the npm upgrade step from the release workflow.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 27, 2026

bumpy-frog

The changes in this PR will be included in the next version bump.

patch Patch releases

  • @varlock/bumpy 1.10.0 → 1.10.1

Bump files in this PR

Click here if you want to add another bump file to this PR


This comment is maintained by bumpy.

The draft release functions (createDraftRelease, updateReleaseBody,
finalizeRelease, deleteRelease) were not using withReleaseToken, so
GitHub release events were created with GITHUB_TOKEN which doesn't
trigger downstream workflows like on-release.yaml.
@theoephraim theoephraim changed the title fix: disable npm staged publishing fix: use BUMPY_GH_TOKEN for draft release operations May 27, 2026
@theoephraim theoephraim merged commit 2f5e859 into main May 27, 2026
2 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