Skip to content

fix(ci): create GitHub release after npm publish - #3394

Open
margaretjgu wants to merge 2 commits into
mainfrom
fix/npm-publish-github-release
Open

fix(ci): create GitHub release after npm publish#3394
margaretjgu wants to merge 2 commits into
mainfrom
fix/npm-publish-github-release

Conversation

@margaretjgu

Copy link
Copy Markdown
Member

The publish workflow was missing the GitHub release creation step. Adds it after npm publish, using the ephemeral vault token. Also bumps contents permission from read to write which is required to create releases.

@margaretjgu
margaretjgu requested a review from JoshMock July 30, 2026 16:56

@JoshMock JoshMock left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The GH release creation step was removed now that we have an upstream "create release" workflow in the team repo that should trigger this downstream npm publish. If it does need to be reinstated, can we add a condition that it only runs when this workflow was triggered by a workflow_dispatch?

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