From: Simon Glass <simon.glass@canonical.com> This series addresses pylint warnings in the buildman board modules, achieving a 10/10 pylint score. For board.py, unavoidable warnings (too-many-instance-attributes, too-few-public-methods, too-many-arguments) are suppressed since the Board class is a legitimate data container. For boards.py, the series: - Fixes line-too-long and missing return documentation warnings - Refactors complex functions by extracting helper methods to reduce too-many-branches and too-many-locals warnings - Adds missing parameter documentation - Suppresses remaining unavoidable warnings Simon Glass (12): buildman: Suppress pylint warnings in board.py buildman: Fix pylint warnings in boards.py buildman: Extract arch fixups from scan() into separate function buildman: Use specific exception type in _fixup_arch() buildman: Extract tag parsing from parse() into _parse_tag() buildman: Extract defconfig loading from scan() into _load_defconfig() buildman: Extract target checking from scan() into _check_targets() buildman: Extract F: and N: tag handling in parse_file() into methods buildman: Split _start_defconfig_scans() into two functions buildman: Extract helper functions from output_is_new() buildman: Extract _check_board() from select_boards() buildman: Fix remaining pylint warnings in boards.py tools/buildman/board.py | 18 +- tools/buildman/boards.py | 565 ++++++++++++++++++++++++++------------- 2 files changed, 383 insertions(+), 200 deletions(-) -- 2.43.0 base-commit: 6cff0c8a556d74547b5a24ec9dad0614a8b9d387 branch: bmk