docs: rework the READMEs to the org layout and fix the install commands - #464
shenxianpeng wants to merge 5 commits into
Conversation
Follow the layout the other cpp-linter READMEs move to. The root README gets four badges (release candidate, ci, coverage, part of), the title cpp-linter-rs and a link line; the package READMEs get the link line. The install commands failed or installed something else: - cargo install and cargo binstall pick 2.0.0-rc9 without --version, which has no bin feature; pin 2.0.0-rc.23. - pip install cpp-linter installs the Python package 1.14.1, and the TestPyPI command needs --pre. - npm's latest tag is the deprecated rc2; install @next. --tidy-review and --format-review were replaced by --pr-review (#377), so the example links to --pr-review, and docs/docs/index.md defines it. Drop the crates.io, PyPI and npm badges that showed rc9, 1.14.1 or rc2, fix the cli.html links (404) in the package READMEs, and use the website's release-candidate wording in the warning.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe README files update v2 release-candidate installation guidance, package availability notes, and project links. The main README and documentation index replace references to separate tidy and format review options with the unified ChangesRelease-candidate documentation
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The updated installation and review-option guidance matches the repository’s local configuration. No concrete regression from these documentation changes warrants holding the PR. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 4 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #464 +/- ##
=======================================
Coverage 92.74% 92.74%
=======================================
Files 23 23
Lines 3859 3859
=======================================
Hits 3579 3579
Misses 280 280 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| See also the [CLI document hosted on GitHub][gh-pages]. | ||
|
|
||
| [gh-pages]: https://cpp-linter.github.io/cpp-linter-rs/cli.html | ||
| [crates-io-badge]: https://img.shields.io/crates/v/cpp-linter | ||
| [crates-io-link]: https://crates.io/crates/cpp-linter | ||
| [docs-badge]: https://img.shields.io/docsrs/cpp-linter | ||
| [docs-link]: https://docs.rs/cpp-linter | ||
| [changelog-badge]: https://img.shields.io/badge/keep_a_change_log-v1.1.0-ffec3d | ||
| [changelog-link]: https://github.com/cpp-linter/cpp-linter-rs/blob/main/cpp-linter/CHANGELOG.md | ||
| [gh-pages]: https://cpp-linter.github.io/cpp-linter-rs/cli/ |
There was a problem hiding this comment.
| See also the [CLI document hosted on GitHub][gh-pages]. | |
| [gh-pages]: https://cpp-linter.github.io/cpp-linter-rs/cli.html | |
| [crates-io-badge]: https://img.shields.io/crates/v/cpp-linter | |
| [crates-io-link]: https://crates.io/crates/cpp-linter | |
| [docs-badge]: https://img.shields.io/docsrs/cpp-linter | |
| [docs-link]: https://docs.rs/cpp-linter | |
| [changelog-badge]: https://img.shields.io/badge/keep_a_change_log-v1.1.0-ffec3d | |
| [changelog-link]: https://github.com/cpp-linter/cpp-linter-rs/blob/main/cpp-linter/CHANGELOG.md | |
| [gh-pages]: https://cpp-linter.github.io/cpp-linter-rs/cli/ | |
| See also the [CLI document hosted on GitHub][cli-doc]. | |
| [cli-doc]: https://cpp-linter.github.io/cpp-linter-rs/cli/ |
Co-authored-by: Brendan <2bndy5@gmail.com>
Co-authored-by: Brendan <2bndy5@gmail.com>
Co-authored-by: Brendan <2bndy5@gmail.com>
Co-authored-by: Brendan <2bndy5@gmail.com>

Part of aligning the READMEs across the org (same layout as cpp-linter/cpp-linter-hooks#289). Every install command was run today.
Install commands
cargo install cpp-linter --features bin*. Any range picks2.0.0-rc9(semver sorts it afterrc.23), which has nobinfeature--version 2.0.0-rc.23cargo binstall cpp-linter--version 2.0.0-rc.23pip install cpp-linter--pre(fails without it on pip 26.2.1 and 25.2)npm install @cpp-linter/cpp-linterlatestis the deprecated rc2@cpp-linter/cpp-linter@next(rc.21)The
rc.23pin has to be bumped by hand at each crate release, sincebump-n-release.nudoes not touch READMEs.README.md
cpp-linter-rs.run-dev-tests.yml), coverage, part of cpp-linter. The five build badges are gone.nextstay, in the same colors.--tidy-review/--format-reviewlinks. It now uses--pr-review(deprecate: replace--tidy-review/--format-reviewwith--pr-review#377), and all 7 images load.## Installbecomes## Quick start, with the content unchanged. "Have question or feedback?" becomes## Contributingand links CONTRIBUTING.md.Package READMEs
cli.htmllinks (404) now point to/cli/.cpp-linter/README.md(crates.io): typos and "Github" spellings are fixed. The CHANGELOG badge is removed; restore it if you want it on crates.io.bindings/python/README.md: the TestPyPI command gets--pre, andcpp-linter -helpbecomes--help.bindings/node/README.md: the install command uses@next.docs/docs/index.md
The
[tidy-review]and[format-review]definitions become[pr-review]. Without this the docs home page would print[--pr-review][pr-review]as text.Checked
readme_rendererpasses for the Python binding README.Outside this PR
latest: it still points to the deprecated rc2. An npm owner can runnpm dist-tag add @cpp-linter/cpp-linter@2.0.0-rc.21 latest.Summary by CodeRabbit
--pr-reviewand refreshed CLI documentation links.