docs: rework the README to the org layout and move required tools to the docs - #493
shenxianpeng wants to merge 1 commit into
Conversation
…the docs Follow the layout the other cpp-linter READMEs move to: name, three badges, one sentence, a link line, then Quick start, Usage, Example, Supported runners, Used by, Contributing and License. The Quick start is the website's Get started workflow. The thread comment step from #491 stays under Usage, unchanged. Move "Required tools installed" to a new docs page, Required Tools, word for word except for the install order: clang-tools 1.3.0 tries the static binaries first and falls back to the PyPI wheels, not the other way round. The README keeps a short Supported runners section. docs/index.md never defined skip-doc, so the docs home page printed "[skip instruction][skip-doc]" as text. The README now defines it inside the included range.
|
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: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review. WalkthroughThe README now provides a project overview, quick-start and usage examples, supported-runner information, and contribution guidance. A new required-tools guide describes tool installation behavior and runner prerequisites, and is linked from the documentation index and navigation. ChangesProject and tool documentation
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change reorganizes the README and adds a Required Tools page linked from the docs index and navigation. It does not change runtime behavior, and no merge-blocking issue was found. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 3 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 |
Part of aligning the READMEs across the org (same layout as cpp-linter/cpp-linter-hooks#289). The README is also the docs home page (between
README-startandREADME-end) and the Marketplace listing.README
Header
cpp-linter-action.self-test.yml, the workflow that runs the action. The marketplace, cpp-linter.yml, MkDocs Deploy and hub badges are gone.Quick start
format-review) and its four bullets, word for word.Moved: Required tools
docs/required-tools.md, added to the nav after Permissions.pyproject.toml) tries the static binaries first and falls back to the PyPI wheels, not the other way round (clang_tools/main.py, theinstallbranch). The README keeps a short## Supported runnerssection that links to the page.Used by
<p align>HTML. The list ends with a link to the showcase.Removed
## Contributing.Docs home page fix
docs/index.mdnever definedskip-doc, so the live docs home page prints[skip instruction][skip-doc]as text. The README now definesskip-docandgh-container-syntaxinside the included range.docs/index.mdgains one line,[tools-doc]: required-tools.md.Checked
uv run mkdocs build --strictpasses.readme_rendererrenders the README without warnings.Summary by CodeRabbit