clang-tidy
.
doc: Remove build instruction for running clang-tidy
#32662
pull
hebasto
wants to merge
1
commits into
bitcoin:master
from
hebasto:250602-doc-clang-tidy
changing
1
files
+0 −1
-
hebasto commented at 12:50 pm on June 2, 2025: memberOne of the benefits of using a compilation database, which is available after the CMake build system generation step, is that it is not necessary to actually build the code in order to run
-
doc: Remove build instruction for running `clang-tidy`
One of the benefits of using a compilation database, which is available after the CMake build system generation step, is that it is not necessary to actually build the code in order to run `clang-tidy`.
-
hebasto added the label Docs on Jun 2, 2025
-
DrahtBot commented at 12:50 pm on June 2, 2025: contributor
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32662.
Reviews
See the guideline for information on the review process.
Type Reviewers ACK TheCharlatan, janb84 If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
LLM Linter (✨ experimental)
Possible typos and grammar issues:
- “denoised of errors” → “denoised, with errors removed” [“denoised of” is not idiomatic English]
drahtbot_id_4_m
-
TheCharlatan approved
-
TheCharlatan commented at 1:43 pm on June 2, 2025: contributorACK 4b1b36acb48fab133ca4a3241148fa9683915874
-
janb84 commented at 1:44 pm on June 2, 2025: contributor
ACK 4b1b36acb48fab133ca4a3241148fa9683915874
Checked both outcomes, the “old” way and the new way, both outcomes are the equal.(the diff was harder to check due to running it in multithreaded mode, therefor the files where out of order)
-
fanquake merged this on Jun 2, 2025
-
fanquake closed this on Jun 2, 2025
-
hebasto deleted the branch on Jun 2, 2025
-
maflcko commented at 1:37 pm on June 3, 2025: member
Shouldn’t this be excluded in the tidy CI task as well?
Also, would be good to mention that this will now print errors about the missing auto-generated files.
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-06-15 06:13 UTC
More mirrored repositories can be found on mirror.b10c.me