Fix CHANGELOG duplicates introduced by #3136 - #3162
Conversation
PR newton-physics#3136 was merged from a branch cut before the 1.3.0 release. The merge re-inserted that branch's stale [Unreleased] entries as an 18-line duplicate block in the now-released [1.3.0] Fixed section, where each fix already appears with its canonical PR reference. Remove the duplicated block so [1.3.0] matches its released state, and add the missing (newton-physics#3136) reference to the genuine new entry in the [Unreleased] section.
📝 WalkthroughWalkthroughThis PR updates ChangesChangelog Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Line 16: Changelog entry wording is incomplete because de-duplication retained
the SensorTiledCamera Unreleased [Fixed] entry with reference (`#3088`) while the
objective only mentions (`#3136`); update the CHANGELOG.md Unreleased section to
mention both fixes by adjusting the objective text to include SensorTiledCamera
and (`#3088`) alongside the mesh inertia fix (`#3136`), and ensure the note about
removing the 18-line duplicate from [1.3.0] remains intact so entries for
SensorTiledCamera, [Fixed], (`#3088`), (`#3136`), and [1.3.0] are all correctly
represented.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Description
PR #3136 was merged from a branch cut before the 1.3.0 release. The merge re-inserted that branch's stale
[Unreleased]entries as an 18-line duplicate block in the now-released[1.3.0]### Fixedsection, where each fix already appears with its canonical PR reference.This removes the duplicated block so
[1.3.0]matches its released state, and adds the missing(#3136)reference to the genuine new entry in[Unreleased]. The result is byte-identical tomainimmediately before the #3136 merge, plus only that one legitimate entry.Checklist
CHANGELOG.mdhas been updated (if user-facing change)Test plan
Docs-only change. Verified the diff against pre-merge
maincontains only the single(#3136)line addition, and that no[1.3.0]entry bodies remain duplicated.Summary by CodeRabbit