Skip to content

fix(release): only publish alpha and production releases to NPM#218

Merged
dkoo merged 2 commits intotrunkfrom
fix/only-publish-alpha-release-to-npm
Oct 16, 2025
Merged

fix(release): only publish alpha and production releases to NPM#218
dkoo merged 2 commits intotrunkfrom
fix/only-publish-alpha-release-to-npm

Conversation

@dkoo
Copy link
Contributor

@dkoo dkoo commented Oct 15, 2025

All Submissions:

Changes proposed in this Pull Request:

Our release CI job will currently publish any tagged release to NPM. This results in pretty messy versioning on NPM. This PR updates the release job to only publish to NPM if the release is an alpha or production release, going forward.

How to test the changes in this Pull Request:

It won't be super easy to test this before it's merged and released to NPM itself, but after that, we can test by pushing a new hotfix/* or epic/* branchname to Newspack Blocks and confirming that the resulting tagged prerelease is NOT published to NPM, but new alpha or release tags ARE published.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@dkoo dkoo requested a review from a team as a code owner October 15, 2025 20:42
@dkoo dkoo self-assigned this Oct 15, 2025
Copy link

@chickenn00dle chickenn00dle left a comment

Choose a reason for hiding this comment

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

Approving with one small nit.

@dkoo dkoo requested a review from chickenn00dle October 15, 2025 23:07
@dkoo dkoo merged commit d10dcb5 into trunk Oct 16, 2025
5 checks passed
@dkoo dkoo deleted the fix/only-publish-alpha-release-to-npm branch October 16, 2025 16:06
@matticbot
Copy link

🎉 This PR is included in version 5.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants