Skip to content

Commit a8098f2

Browse files
author
MacroFake
committed
Merge bitcoin/bitcoin#25091: test: Remove extended lint (cppcheck)
efae252 test: Remove extended lint (cppcheck) (laanwj) Pull request description: These are unreferenced in the CI and documentation, and have been since 2019 (see #17549). I'm not sure the cppcheck is worthwhile. It takes a long time to run (I think this is why it isn't in the normal lints), and right now it only appears to find implicit constructors. The list of exceptions is out of date. But if anyone wants to bring it back at any time in the future they can do so from git history (and port it to Python). ACKs for top commit: fanquake: ACK efae252 Tree-SHA512: 1a770b5d20ff1199d0d6bc471ae3d2c3438f0f0b169ce8d2fe73480daf8d3a7146c066b799afc90aa7898982c5fee79c1daca10e16e2bff0a7b38850aedd55b2
2 parents dab18f0 + efae252 commit a8098f2

File tree

2 files changed

+0
-114
lines changed

2 files changed

+0
-114
lines changed

test/lint/extended-lint-all.sh

Lines changed: 0 additions & 26 deletions
This file was deleted.

test/lint/extended-lint-cppcheck.sh

Lines changed: 0 additions & 88 deletions
This file was deleted.

0 commit comments

Comments
 (0)