-
Notifications
You must be signed in to change notification settings - Fork 930
[Release] Version packages #4443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@claude Why does a minor changeset cause the major version number to increase? |
|
Claude finished @felixfeng33's task —— View job I found the answer! This isn't actually a case of a minor changeset causing a major version bump. Here's what's happening: Todo List:
ExplanationThe packages were already bumped to version 50.0.0 before this changeset was added. Looking at the CHANGELOG, I can see that version 50.0.0 already exists with your minor changes listed under it. This appears to be how the workflow operates:
The changeset configuration shows all packages are linked together: "linked": [
[
"platejs",
"@platejs/*",
"@udecode/react*",
"@udecode/cn",
"@udecode/utils"
]
]This means all packages maintain the same major version number. When the decision was made to go to v50, all packages were updated together, regardless of the individual changesets being minor or patch. TL;DR: The major version bump to 50.0.0 was a deliberate decision that happened before your changeset. Your minor changeset is just being included in the v50 release, not causing it. |
|
@claude This PR was automatically created by CI, so did the minor changeset cause the major version bump but why? |
|
Claude finished @felixfeng33's task —— View job Looking into why the minor changeset caused a major version bump to 50.0.0... Todo List:
AnalysisAfter investigating, here's what I found:
"linked": [
[
"platejs",
"@platejs/*",
"@udecode/react*",
"@udecode/cn",
"@udecode/utils"
]
]
ConclusionThe major version bump to 50.0.0 appears to be a deliberate decision by the maintainers rather than something triggered by your minor changeset. This likely happened for one of these reasons:
Your minor changeset didn't cause the major bump - it's just being included in the v50 release. The major version bump was a separate decision that happened to coincide with your changes being released. |
ae3ebd8 to
9f1ce62
Compare
9f1ce62 to
b683700
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@platejs/[email protected]
Patch Changes
#4441 by @delijah – Expose mimeType to plugin parser functions
#4431 by @felixfeng33 – Added comprehensive copy functionality and view editor support for static rendering.
New Components:
PlateViewcomponent for static editor rendering with copy supportusePlateViewEditorhook for creating memoized static editorsStatic Editor Enhancements:
withStaticHOC to enhance editors with static rendering capabilitiesViewPluginthat enables copy operations in static editorsgetStaticPluginsto configure plugins for static renderingonCopyhandler that properly serializes content withx-slate-fragmentformatNew Utilities:
getSelectedDomBlocksto extract selected DOM elements with Slate metadatagetSelectedDomNodeto get DOM nodes from browser selectionisSelectOutsideto check if selection is outside editor boundsgetPlainTextto recursively extract plain text from DOM nodesThis enables seamless copy operations from static Plate editors, allowing content to be pasted into other Slate editors while preserving rich formatting and structure.
[email protected]
@platejs/[email protected]