codespell
as a requirement.
docs: Document lint tests #14526
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:revive-document-lint-tests changing 1 files +25 −4-
fanquake commented at 5:10 am on October 20, 2018: memberReplaces #13708. Fixed the macOS nit, and added
-
docs: Document lint tests 8640631ebc
-
fanquake added the label Docs on Oct 20, 2018
-
in test/README.md:188 in 8640631ebc
180@@ -180,6 +181,26 @@ Note: gdb attach step may require `sudo` 181 Util tests can be run locally by running `test/util/bitcoin-util-test.py`. 182 Use the `-v` option for verbose output. 183 184+### Lint tests 185+ 186+#### Dependencies 187+ 188+The lint tests require codespell and flake8. To install: `pip3 install codespell flake8`.
promag commented at 8:47 am on October 20, 2018:Just noting that I think it should not specify versions, this way new lint errors can be detected and the source fixed accordingly. -
practicalswift commented at 7:12 pm on October 20, 2018: contributor
Nice!
ACK 8640631ebcac23c69b657e09e7206ea6a8b35d19
-
MarcoFalke merged this on Oct 21, 2018
-
MarcoFalke closed this on Oct 21, 2018
-
MarcoFalke referenced this in commit 73cddb36a8 on Oct 21, 2018
-
fanquake deleted the branch on Nov 21, 2018
-
PastaPastaPasta referenced this in commit dc425c3fea on Jul 17, 2021
-
DrahtBot locked this on Sep 8, 2021
fanquake
promag
practicalswift
Labels
Docs