chore: release the removed web flag as a minor bump - #218
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe changeset changes ChangesRelease metadata
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized release-metadata change preserves the existing changelog message while changing the next version from 2.0.0 to 1.4.0; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description includes all required sections, explains why the changeset changes from major to minor, describes the user impact, and records checklist status. The missing issue link is explained because this is a release-policy change. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) Comment |
commit: |
Related Issue
None — release version policy.
Problem
The pending release PR (#193) resolves to
@pymodel/pythinker-code@2.0.0becausedrop-allow-remote-terminals.mdcarries amajorbump. We do not want to spend the 2.0.0 version on this change.What changed
Changed that changeset from
majortominor. The release PR regenerates from.changeset/onmain, so the next release becomes1.4.0instead of2.0.0. The changelog sentence stays the same, so users still read that the flag is gone.Note the trade-off: removing a CLI flag under a minor bump gives users no major-version signal. Anyone passing
--allow-remote-terminalstopythinker webstill gets a clear error, and PTY terminal routes remain available on loopback binds.Checklist
/approve).gen-changesetsskill, or this PR needs no changeset. [skip changeset]gen-docsskill, or this PR needs no doc update.Summary by CodeRabbit
Breaking Changes
--allow-remote-terminalsoption.Release