doc: add entries to changelog#35
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 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.
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>
a4a1dc5 to
3c74663
Compare
Add commits (and their associated dates) to the changelog that were written by Zac Dover on behalf of Clyso.
Summary by CodeRabbit