Skip to content

Do npm publishing in release.yml#642

Merged
jackkleeman merged 1 commit intomainfrom
publish-in-release
Feb 5, 2026
Merged

Do npm publishing in release.yml#642
jackkleeman merged 1 commit intomainfrom
publish-in-release

Conversation

@jackkleeman
Copy link
Contributor

Changes (compared to before I started down this path):

  • Merge release.yml, publish.yml, and snapshot publishing from test.yml into a single release.yml workflow (npm publishing must happen from a single file for trusted publishing)
  • Remove publish.yml and manual-publish.yml (manual trigger now via workflow_dispatch on release.yml)
  • release.yml now calls test.yml via workflow_call before publishing, ensuring tests pass first
  • test.yml no longer triggers on push to main (only PRs, workflow_call, and manual dispatch)

@jackkleeman jackkleeman requested a review from nikrooz February 5, 2026 22:54
@jackkleeman jackkleeman merged commit ec48367 into main Feb 5, 2026
8 checks passed
@jackkleeman jackkleeman deleted the publish-in-release branch February 5, 2026 22:59
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