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
  1. MarcoFalke commented at 2:45 PM on April 4, 2022: member

    No description provided.

  2. MarcoFalke cross-referenced this on Apr 4, 2022 from issue test, lint: check bash version before executing codespell by brunoerg
  3. laanwj commented at 2:53 PM on April 4, 2022: member

    Yess, concept ACK.

  4. Move lint script and data file to avoid lint- prefix
    This is needed for the next commit
    fa82e890e7
  5. lint: Start to use py lint scripts fae211c0ae
  6. fanquake commented at 3:00 PM on April 4, 2022: member

    Concept ACK

  7. MarcoFalke force-pushed on Apr 4, 2022
  8. DrahtBot added the label Tests on Apr 4, 2022
  9. fjahr cross-referenced this on Apr 4, 2022 from issue lint: convert spellchecking lint test to python by fjahr
  10. fjahr commented at 10:30 PM on April 4, 2022: contributor

    tACK fae211c0ae0dd90876a3390eb21449b7b0bb45c4

    Reviewed code and took it for a spin with #24766. Didn't see any issues.

    mini-nit: lint-files.sh is 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

  11. DrahtBot commented at 11:57 PM on April 4, 2022: contributor

    <!--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.

  12. MarcoFalke merged this on Apr 5, 2022
  13. MarcoFalke closed this on Apr 5, 2022
  14. MarcoFalke deleted the branch on Apr 5, 2022
  15. DrahtBot cross-referenced this on Apr 5, 2022 from issue test: Refactor subtree exclusion in lint tests by maxraustin
  16. sidhujag referenced this in commit 1abfcd11d7 on Apr 5, 2022
  17. bitcoin locked this on Apr 5, 2023
Contributors
MarcoFalkelaanwjfanquakefjahrDrahtBot
Labels
Tests
Linked (view graph)
#24610 test, lint: check bash version before executing codespell#24766 lint: convert spellchecking lint test to python
github-metadata-mirror

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: 2026-05-20 06:53 UTC