[PATCH 0/6] Add a pylint check

Simon Glass sjg at chromium.org
Mon Nov 22 04:48:35 CET 2021


U-Boot has about 60k lines of Python code. This is quite small relative
to the total code size, but it is still important to keep the code style
consistent.

This series allows pylint to be run on Python files within U-Boot to
detect any changes that reduce the current pylint ‘score’ of any modules.
It works by comparing the current state against a base state kept in a
file.

The check is invoked with:

   make pylint

This should gradually help to improve the code.

The series also tidies up a few minor nits to produce better pylint
output.


Simon Glass (6):
  .gitignore: Ignore any html coverage directory
  dtoc: Fix up a code comment that confuses pylint
  patman: Update the list of modules
  tools: Add init files for Python tools
  Makefile: Add a pylint checker to the build
  Azure/GitLab CI: Add the pylint checker

 .azure-pipelines.yml       |  17 +++
 .gitignore                 |   7 ++
 .gitlab-ci.yml             |  12 +++
 Makefile                   |  45 +++++++-
 doc/develop/index.rst      |   8 ++
 doc/develop/python_cq.rst  |  80 ++++++++++++++
 scripts/pylint.base        | 211 +++++++++++++++++++++++++++++++++++++
 tools/binman/__init__.py   |   0
 tools/buildman/__init__.py |   0
 tools/dtoc/__init__.py     |   0
 tools/dtoc/dtb_platdata.py |   2 +-
 tools/patman/__init__.py   |   9 +-
 12 files changed, 386 insertions(+), 5 deletions(-)
 create mode 100644 doc/develop/python_cq.rst
 create mode 100644 scripts/pylint.base
 create mode 100644 tools/binman/__init__.py
 create mode 100644 tools/buildman/__init__.py
 create mode 100644 tools/dtoc/__init__.py

-- 
2.34.0.rc2.393.gf8c9666880-goog



More information about the U-Boot mailing list