No description provided.
lint: Start to use py lint scripts #24762
pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2204-lint-with-py-🥂 changing 7 files +8 −15-
MarcoFalke commented at 2:45 PM on April 4, 2022: member
-
laanwj commented at 2:53 PM on April 4, 2022: member
Yess, concept ACK.
-
fa82e890e7
Move lint script and data file to avoid lint- prefix
This is needed for the next commit
-
lint: Start to use py lint scripts fae211c0ae
-
fanquake commented at 3:00 PM on April 4, 2022: member
Concept ACK
- MarcoFalke force-pushed on Apr 4, 2022
- DrahtBot added the label Tests on Apr 4, 2022
-
fjahr commented at 10:30 PM on April 4, 2022: member
tACK fae211c0ae0dd90876a3390eb21449b7b0bb45c4
Reviewed code and took it for a spin with #24766. Didn't see any issues.
mini-nit:
lint-files.shis still mentioned in the test readme but I can also take care of that in the follow-up https://github.com/bitcoin/bitcoin/pull/24766/commits/494e9918ee4311ba621f4ae965aa1bdea3b08f55 -
DrahtBot commented at 11:57 PM on April 4, 2022: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #24435 (test: Refactor subtree exclusion in lint tests by maxraustin)
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.
- MarcoFalke merged this on Apr 5, 2022
- MarcoFalke closed this on Apr 5, 2022
- MarcoFalke deleted the branch on Apr 5, 2022
- sidhujag referenced this in commit 1abfcd11d7 on Apr 5, 2022
- DrahtBot locked this on Apr 5, 2023
