doc: add entries to changelog#35
Conversation
📝 WalkthroughWalkthroughUpdated changelog table in ChangesDocumentation Table Enhancement
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 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: 2
🤖 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 `@docs/changelog/index.md`:
- Line 12: The changelog entry has an incorrect year: the date string
"2025-03-12" in the table row should be "2026-03-12" to keep chronological order
with surrounding entries; update the date literal in the docs/changelog index
table (the row containing "Add Arbiter documentation | 26891975") to
"2026-03-12".
- Around line 17-18: Remove the duplicate changelog entry that appears twice ("|
2026-03-31 | Add CephFS summary to arch | fd962649 |"); keep a single
instance of that line in docs/changelog/index.md so the changelog is
deduplicated and consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
f786467 to
a4a1dc5
Compare
senolcolak
left a comment
There was a problem hiding this comment.
Thanks for the changes Zac. I think this is more of a commit list than a changelog.
Could we make it a bit more changelog like, we can group related documentation changes and describing the actual purpose/impact? we will have less entries on the list but will be more informative,
the CephFS entry appears twice with the same hash.
The git hashes are useful, but the change descriptions should be a bit more meaningful for reviewers/readers.
a4a1dc5 to
3c74663
Compare
Add commits (and their associated dates) to the changelog that were written by Zac Dover on behalf of Clyso. Signed-off-by: Zac Dover <zac.dover@clyso.com>
3c74663 to
fe75144
Compare
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 `@docs/changelog/index.md`:
- Around line 16-18: The changelog file contains a duplicate entry for CephFS
with git hash fd962649 that appears twice with identical content and the same
date of 2026-03-31. Locate both instances of this CephFS changelog entry and
remove one of the duplicates to ensure each changelog entry appears only once in
the file.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
senolcolak
left a comment
There was a problem hiding this comment.
looks clean and to the point, thanks Zac
Add commits (and their associated dates) to the changelog that were written by Zac Dover on behalf of Clyso.
Summary by CodeRabbit