mbox series

[Concept,00/12] buildman: Fix pylint warnings in board.py and boards.py

Message ID 20260103203243.3727493-1-sjg@u-boot.org
Headers
Series buildman: Fix pylint warnings in board.py and boards.py |

Message

Simon Glass Jan. 3, 2026, 8:32 p.m. UTC 
  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(-)