Skip to content

Fix mixed changeset blocking npm publish - #95

Merged
martin-henz merged 1 commit into
mainfrom
fix/stepper-hover-changeset
Aug 7, 2026
Merged

Fix mixed changeset blocking npm publish#95
martin-henz merged 1 commit into
mainfrom
fix/stepper-hover-changeset

Conversation

@martin-henz

Copy link
Copy Markdown
Member

Summary

  • .changeset/stepper-hover-text-popover.md declared a patch bump for both @sourceacademy/common-stepper and @sourceacademy/web-stepper.
  • web-stepper is in .changeset/config.json's ignore list (bundled host plugin, not published to npm — same treatment as web-data-visualizer), so changesets rejects any changeset that mixes an ignored and non-ignored package.
  • This was hard-blocking yarn changeset version/publish on every push to main since the "Version Packages" PR (Version Packages #91) merged — nothing has published to npm.

Fix: drop web-stepper from this changeset, keeping the common-stepper bump.

Test plan

  • CI build/tests pass
  • Merge, then confirm the next on_push run's "Create Release Pull Request or Publish to npm" step succeeds (or opens a fresh Version Packages PR)

web-stepper is in .changeset/config.json's ignore list (bundled host
plugin, not a published library, matching web-data-visualizer's
treatment), so a changeset can't declare a bump for it alongside a
non-ignored package. This mix was blocking `yarn changeset version`
on main, so nothing has been publishing to npm.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FavSXvMgNNKtZZVeXBfDZe
@changeset-bot

changeset-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 44c5328

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "@sourceacademy/web-stepper" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch
The package or glob expression "@sourceacademy/web-data-visualizer" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch

@martin-henz
martin-henz merged commit 229d826 into main Aug 7, 2026
2 checks passed
@martin-henz
martin-henz deleted the fix/stepper-hover-changeset branch August 7, 2026 03:12
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