[PATCH 0/9] Enable the pylint checker in CI

Simon Glass sjg at chromium.org
Fri Feb 11 21:23:17 CET 2022


This series adds a new errors-only pylint check and adds it to the CI
systems.

It also fixes the current errors in the U-Boot Python code, disabling
errors where it seems necessary.

A small patch to buildman allows it to build sandbox without any changes
to the default config file.


Simon Glass (9):
  patman: Correct pylint errors
  buildman: Correct pylint errors
  dtoc: Correct pylint errors
  binman: Correct pylint errors
  moveconfig: Correct pylint errors
  test: Correct pylint errors
  Makefile: Add a way to check for pylint errors
  buildman: Update default config to build for sandbox
  Azure/GitLab CI: Add the pylint checker

 .azure-pipelines.yml                        | 22 +++++++++++++++++++++
 .gitlab-ci.yml                              | 16 +++++++++++++++
 Makefile                                    | 10 ++++++++--
 doc/develop/python_cq.rst                   | 11 +++++++++++
 test/py/tests/test_android/test_avb.py      |  2 +-
 test/py/tests/test_bind.py                  |  8 ++++----
 test/py/tests/vboot_evil.py                 |  3 ++-
 test/py/u_boot_console_base.py              |  8 ++++++++
 tools/binman/cmdline.py                     |  2 +-
 tools/binman/control.py                     |  6 ++++++
 tools/binman/elf_test.py                    | 13 ++++++------
 tools/binman/entry.py                       |  2 ++
 tools/binman/entry_test.py                  |  7 ++-----
 tools/binman/etype/blob_dtb.py              |  3 +++
 tools/binman/etype/blob_phase.py            |  3 +++
 tools/binman/etype/cbfs.py                  |  3 +++
 tools/binman/etype/fdtmap.py                |  5 +++++
 tools/binman/etype/files.py                 |  2 ++
 tools/binman/etype/section.py               |  1 +
 tools/binman/etype/u_boot_dtb_with_ucode.py |  3 +++
 tools/binman/ftest.py                       | 21 ++------------------
 tools/buildman/bsettings.py                 |  1 +
 tools/buildman/builder.py                   |  2 +-
 tools/buildman/cfgutil.py                   |  6 +++---
 tools/buildman/func_test.py                 |  6 ------
 tools/buildman/kconfiglib.py                |  1 +
 tools/buildman/main.py                      |  4 ++--
 tools/concurrencytest/__init__.py           |  0
 tools/dtoc/test_dtoc.py                     |  6 +++---
 tools/dtoc/test_fdt.py                      |  6 +++---
 tools/moveconfig.py                         |  5 +----
 tools/patman/checkpatch.py                  |  4 +++-
 tools/patman/command.py                     |  8 +-------
 tools/patman/commit.py                      |  2 +-
 tools/patman/cros_subprocess.py             | 18 ++++-------------
 tools/patman/func_test.py                   | 10 ++++++++++
 tools/patman/patchstream.py                 |  2 +-
 tools/patman/series.py                      |  3 +--
 tools/patman/settings.py                    |  4 ++--
 tools/patman/tools.py                       | 10 +++++-----
 40 files changed, 154 insertions(+), 95 deletions(-)
 create mode 100644 tools/concurrencytest/__init__.py

-- 
2.35.1.265.g69c8d7142f-goog



More information about the U-Boot mailing list