docs: correct the 8.0.0 changelog entry - #760
Conversation
8.0.0 shipped with a breaking-change notice describing a clustering format change that is not in the release. #739 introduced it and #746 reverted it before release, but release-please does not pair a revert with its original, so the BREAKING CHANGE footer both drove the major bump and rendered into the notes. Replaces the notice with a note explaining why the major carries no breaking change, and moves #739 from Code Refactoring to Reverts. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 27 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Problem
8.0.0 shipped with a
⚠ BREAKING CHANGESnotice describing a clustering format change that is not in the release.#739 introduced that change and #746 reverted it before release. Release-please does not pair a revert with its original, so the
BREAKING CHANGE:footer both drove the major version bump and rendered into the published notes.The release is in fact non-breaking. Verified by loading artifacts from released versions against this code:
Serialized clustering is unchanged from 7.2.1, and 6.x files that stopped loading under 7.0 load again as of #757 — so 8.0.0 is more compatible than its predecessor, not less.
Change
CHANGELOG.mdonly, for the already-published 8.0.0 section:Code RefactoringtoReverts, stating it was reverted before releaseUses the
!!! noteadmonition already used elsewhere in this file, so it renders in the docs site.Typed
docs:so it does not itself trigger a release. The v8.0.0 GitHub release notes are being corrected separately, in place.Follow-up
8.0.1 should pick up #755 (raises the
tsam_xarrayfloor to 0.6.5). Without it, anyone resolving to 0.6.5 gets acluster_weightsdeprecation warning — noisy at runtime, and a hard failure under-W error.🤖 Generated with Claude Code