Skip to content

Fix changelog link in README.rst#433

Merged
aaronliu0130 merged 1 commit intocpplint:developfrom
peternewman:patch-1
Apr 19, 2026
Merged

Fix changelog link in README.rst#433
aaronliu0130 merged 1 commit intocpplint:developfrom
peternewman:patch-1

Conversation

@peternewman
Copy link
Copy Markdown
Contributor

@peternewman peternewman commented Apr 18, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Documentation
    • Updated documentation links to reference the develop branch changelog for the latest release information.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 18, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 705dac76-d9ff-4ee5-8fba-35bdaba1773e

📥 Commits

Reviewing files that changed from the base of the PR and between 9726877 and bdd9c8c.

📒 Files selected for processing (1)
  • README.rst
📝 Walkthrough

Walkthrough

Updated a documentation link in README.rst to reference the develop branch version of CHANGELOG.rst instead of the master branch. No functional or logic changes were introduced.

Changes

Cohort / File(s) Summary
Documentation Link Update
README.rst		 Updated "Changes" section hyperlink to point to develop branch CHANGELOG instead of master branch.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link through the branches we follow with care,
From master to develop, we're pointing right there!
One hop, one update, so simple and bright,
Documentation flows smooth with branches set right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix changelog link in README.rst' directly and clearly describes the main change: updating a changelog link in the README file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

aaronliu0130
aaronliu0130 approved these changes Apr 19, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@aaronliu0130 aaronliu0130 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh dear

@aaronliu0130 aaronliu0130 merged commit 039713b into cpplint:develop Apr 19, 2026
8 checks passed
@aaronliu0130
Copy link
Copy Markdown
Member

aaronliu0130 commented Apr 19, 2026

Thank you!

@peternewman peternewman deleted the patch-1 branch April 19, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aaronliu0130 aaronliu0130 aaronliu0130 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@peternewman @aaronliu0130