From: Simon Glass <simon.glass@canonical.com> This series addresses pylint warnings and improves code quality in buildman's test.py file. Changes include: - Fix formatting issues (indentation, long lines, f-strings) - Add missing docstrings and fix existing ones - Convert CamelCase to snake_case for consistency - Rename constants to UPPER_CASE - Split large test classes into smaller, focused classes - Refactor complex methods to reduce branch count - Disable unavoidable pylint warnings with descriptive comments Simon Glass (22): buildman: Fix indentation and semicolon in test.py buildman: Add __init__() to Options class in test.py buildman: Convert to f-strings in test.py buildman: Fix import order in test.py buildman: Convert CamelCase to snake_case in test.py buildman: Add missing docstrings in test.py buildman: Fix singleton comparisons in test.py buildman: Use tools.write_file() in test.py buildman: Use set comprehension in test.py buildman: Fix docstring in add_line_prefix() buildman: Remove unused variables in test.py buildman: Fix implicit string concatenation in test.py buildman: Mark unused arguments in test.py buildman: Disable protected-access warning in test.py buildman: Rename commit parameter to avoid shadowing in test.py buildman: Rename constants to UPPER_CASE in test.py buildman: Shorten long lines in test.py buildman: Disable pylint warnings for Options in test.py buildman: Split TestBuild into multiple classes in test.py buildman: Refactor _check_output() in test.py buildman: Split test_process_limit() into two tests in test.py buildman: Disable some final pylint warnings in test.py tools/buildman/main.py | 6 +- tools/buildman/test.py | 505 ++++++++++++++++++++++++----------------- 2 files changed, 303 insertions(+), 208 deletions(-) -- 2.43.0 base-commit: e92c37debe22e17f39ecaee4304da678ff37cbed branch: bmn