test/lint/lint-includes.sh
to a Python-script as part of the request of #24783. Checked for output-consistency.
lint: Convert lint-includes.sh to Python #24895
pull Kvaciral wants to merge 1 commits into bitcoin:master from Kvaciral:lint-includes-port changing 2 files +179 −103-
Kvaciral commented at 4:24 pm on April 16, 2022: contributorA port of
-
DrahtBot added the label Tests on Apr 16, 2022
-
Kvaciral force-pushed on Apr 16, 2022
-
DrahtBot commented at 4:57 am on April 17, 2022: member
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #22417 (util/system: Close non-std fds when execing slave processes by luke-jr)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
-
Kvaciral force-pushed on Apr 18, 2022
-
Kvaciral force-pushed on Apr 18, 2022
-
lint: Convert lint-includes.sh to Python 67b41678c8
-
Kvaciral force-pushed on Apr 19, 2022
-
KevinMusgrave commented at 3:34 am on April 20, 2022: contributor
Tested ACK 67b41678c8284715c9cf96d6bdbde9a072b7a904
It matches the old output. Example:
0Duplicate include(s) in src/bench/addrman.cpp: 1#include <vector> 2 3Duplicate include(s) in src/bench/base58.cpp: 4#include <array> 5 6The following files #include .cpp files: 7src/bench/base58.cpp:#include <bench/bench.cpp> 8src/bench/bench_bitcoin.cpp:#include <bench/bench.cpp> 9src/bench/bench_bitcoin.cpp:#include <clientversion.cpp> 10 11A new Boost dependency in the form of "boost/algorithm/cats.hpp" appears to have been introduced: 12src/bench/block_assemble.cpp:#include <boost/algorithm/cats.hpp> 13 14A new Boost dependency in the form of "boost/algorithm/dogs.hpp" appears to have been introduced: 15src/bench/bech32.cpp:#include <boost/algorithm/dogs.hpp> 16 17Please use bracket syntax includes ("#include <foo.h>") instead of quote syntax includes: 18src/bench/bech32.cpp:#include "vector" 19src/bench/block_assemble.cpp:#include "crypto/sha256.h"
-
MarcoFalke merged this on Apr 20, 2022
-
MarcoFalke closed this on Apr 20, 2022
-
sidhujag referenced this in commit f38719c942 on Apr 22, 2022
-
DrahtBot locked this on Apr 20, 2023
Kvaciral
DrahtBot
KevinMusgrave
Labels
Tests
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: 2024-11-17 06:12 UTC
More mirrored repositories can be found on mirror.b10c.me