run-clang-tidy uses all available processors by default, so explicitly passing -j $(nproc) is unnecessary.
For example, ci/test/03_test_script.sh doesn't use the -j option.
doc: don't use the -j option for run-clang-tidy #35656
pull fabio-porcedda wants to merge 1 commits into bitcoin:master from fabio-porcedda:doc-clang-tidy changing 1 files +2 −2-
fabio-porcedda commented at 11:04 AM on July 4, 2026: none
- DrahtBot added the label Docs on Jul 4, 2026
-
DrahtBot commented at 11:04 AM on July 4, 2026: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35656.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
149315019a
doc: don't use the -j option for run-clang-tidy
run-clang-tidy uses all available processors by default, so explicitly passing -j $(nproc) is unnecessary. For example, ci/test/03_test_script.sh doesn't use the -j option.
- fabio-porcedda force-pushed on Jul 4, 2026
- fabio-porcedda renamed this:
doc: don't use the -j option for clang-tidy
doc: don't use the -j option for run-clang-tidy
on Jul 4, 2026 -
maflcko commented at 11:20 AM on July 4, 2026: member
thx, but I think it is harmless to leave as-is. in fact, it may be useful to keep, to clarify devs are free to reduce it, and how to reduce it
-
- sedited closed this on Jul 4, 2026
Contributors
Labels