Skip to content

Commit 8f70685

Browse files
MacroFakeknst
authored andcommitted
Merge bitcoin#25288: test: Reliably don't start itself (lint-all.py runs all tests twice)
BACKPORT NOTE: only left-over changes has not been done previously is replacing a mask "lint-*" to "lint-*.py" and it's done f26a496 test: clean up all-lint.py (Martin Leitner-Ankerl) 64d72c4 test: rename lint-all.py to all-lint.py (Martin Leitner-Ankerl) Pull request description: When running `./test/lint/lint-all.py`, the script runs all tests but also calls itself because the comparison with `__file__` doesn't work. Comparing resolved paths gives reliable comparison, and lint-all.py doesn't call itself any more ACKs for top commit: laanwj: Code review ACK f26a496 Tree-SHA512: b44abdd685f7b48a6a9f48e96d97138b635c31c1c7ab543cb5636b5f49690ccd56fa6fec01ae7fcc16af01a613372ee77632f70c32059919b373aa8051953791
1 parent f95ba20 commit 8f70685

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/lint/all-lint.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
exit_code = 0
1717
mod_path = Path(__file__).parent
18-
lints = glob(f"{mod_path}/lint-*")
18+
lints = glob(f"{mod_path}/lint-*.py")
1919
if which("parallel") and which("column"):
2020
logfile = "parallel_out.log"
2121
command = ["parallel", "--jobs", "100%", "--will-cite", "--joblog", logfile, ":::"] + lints

0 commit comments

Comments
 (0)