Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Apr 30, 2022

currently in-tree files like wallet/test/fuzz/coinselection.cpp are missed. Also out-of-tree files like test/data/bip341_wallet_vectors.json.h or qt/moc_qvaluecombobox.cpp are included.

Change the script to only use in-tree files.

Also, change 'python3' to sys.executable.

@laanwj laanwj added the Tests label Apr 30, 2022
@fanquake fanquake requested a review from laanwj May 1, 2022 11:42
@laanwj
Copy link
Member

laanwj commented May 2, 2022

Code review ACK fad0abf

@laanwj laanwj merged commit 037c5e5 into bitcoin:master May 2, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 2, 2022
@maflcko maflcko deleted the 2204-lint-circ-🚄 branch May 3, 2022 08:27
@bitcoin bitcoin locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants