mbox series

[0/6] Add a pylint check

Message ID 20211122034841.304138-1-sjg@chromium.org
Headers show
Series Add a pylint check | expand

Message

Simon Glass Nov. 22, 2021, 3:48 a.m. UTC 
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