docs: fix the quick start, the clang-tidy examples and Used By - #285
Conversation
- The quick start's `--checks='boost-*,...'` was split at each comma by YAML, so clang-tidy got the globs as file names and stopped with "no checks enabled". The whole argument is now quoted. - `--checks=.clang-tidy` does not load .clang-tidy: clang-tidy reads that file by itself and treats the value as one more check glob, which matches nothing. It is removed from all ten examples. - Used By drops Bosch Research (only forks of CONVINCE repositories), Jupyter Xeus (the hook is commented out) and Rancher Sandbox (the repository moved to Kubewarden), and adds doldecomp, HKUST Aerial Robotics and Kubewarden, whose default branches run the hook.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #285 +/- ##
=======================================
Coverage 97.53% 97.53%
=======================================
Files 3 3
Lines 243 243
=======================================
Hits 237 237
Misses 6 6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
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 (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe README updates clang-tidy command examples, including compilation-database options. It also revises the Used By project list and changes the destination of the projects link. ChangesREADME updates
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Merge Risk: ⚪ Minimal · up to No specific behavior or documentation issue requiring resolution before merge is established by the supplied review context. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. 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 |



Checked on 2026-09-26 with pre-commit 4.6 and
rev: v1.6.0in a scratch repository.Quick start. Copied as written, the clang-tidy hook fails. YAML splits
[--checks='boost-*,bugprone-*,...']at the commas, so clang-tidy getsbugprone-*and the rest as file names:Could not auto-detect compilation database for file "bugprone-*", thenError: no checks enabled.Quoting the whole argument fixes it; with the new snippet the hook runs the listed checks.--checks=.clang-tidy. The hook passes--checksto clang-tidy unchanged, and clang-tidy takes the value as a check glob. With clang-tidy 22.1.0 and a.clang-tidyof-*,readability-braces-around-statements,--list-checksprints the same checks with and without the flag, and--dump-configshows.clang-tidyappended toChecks. clang-tidy finds.clang-tidyby itself, so the flag is dropped from all ten examples.Used By.
.pre-commit-config.yamlL76-79).The last line now points to the showcase, since the hub has no full list of organizations.
Summary by CodeRabbit