Just a few checks, which I think make sense:
- Some bugprone checks from upstream
- Some more bugprone checks, which I consider useful
- Some modernize checks, which I consider useful
All checks pass cleanly on the codebase
.clang-tidy: Enable additional checks #380
pull maflcko wants to merge 3 commits into bitcoin-core:master from maflcko:2610-tidy changing 1 files +25 −2-
maflcko commented at 11:08 AM on October 22, 2026: member
-
.clang-tidy: Enable checks from upstream Bitcoin Core 7460e35896
-
.clang-tidy: Enable additional checks not in upstream Bitcoin Core a4fa10754c
-
DrahtBot commented at 11:09 AM on October 22, 2026: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline and AI policy for information on the review process.
Type Reviewers ACK hebasto If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
maflcko commented at 11:22 AM on October 22, 2026: member
Not sure what the openbsd ci failure is, but it seems unrelated, as no source code is changed here, only a config file
-
hebasto commented at 11:25 AM on October 22, 2026: member
Not sure what the openbsd ci failure is, but it seems unrelated...
FWIW, the OpenBSD job has switched to release 8.0.
-
-
.clang-tidy: Enable additional modernize checks fabc490126
- maflcko force-pushed on Oct 28, 2026
- hebasto approved
-
hebasto commented at 1:49 PM on October 30, 2026: member
ACK fabc4901269c0e20974dd41cce068a819252a58c.
