From: Simon Glass <simon.glass@canonical.com> This series cleans up pylint warnings across several buildman modules, bringing them all to 10.00/10: - main.py: Remove dead imports, fix docstrings, fix control flow - bsettings.py: Add docstrings, fix exception handling, use f-strings - test_bsettings.py: Use tools.read_file/write_file helpers - toolchain.py: Rename CamelCase methods to snake_case, add docstrings, document public members, use f-strings, fix various warnings - test_boards.py: Fix long lines, unused variables The toolchain.py changes are split into multiple commits for easier review, as the file required significant updates. Simon Glass (11): buildman: Fix pylint warnings in main.py claude: Drop mention of --no-ext-diff buildman: Fix pylint warnings in bsettings.py buildman: Use tools.read_file/write_file in test_bsettings.py buildman: Rename toolchain methods to snake_case buildman: Fix docstrings in toolchain.py buildman: Make MyHTMLParser.re_arch private buildman: Document all Toolchain public members buildman: Use f-strings in toolchain.py buildman: Fix remaining pylint warnings in toolchain.py buildman: Fix pylint warnings in test_boards.py CLAUDE.md | 1 - tools/buildman/bsettings.py | 60 +++--- tools/buildman/builder.py | 2 +- tools/buildman/builderthread.py | 6 +- tools/buildman/control.py | 16 +- tools/buildman/func_test.py | 6 +- tools/buildman/main.py | 30 ++- tools/buildman/test.py | 90 ++++---- tools/buildman/test_boards.py | 72 ++++--- tools/buildman/test_bsettings.py | 16 +- tools/buildman/toolchain.py | 350 +++++++++++++++++-------------- tools/qconfig.py | 8 +- 12 files changed, 346 insertions(+), 311 deletions(-) -- 2.43.0 base-commit: 37a183535e61f03ad872b085c358bc7fe0a0a5ba branch: bml