diff options
| author | Tom Rini <trini@konsulko.com> | 2022-03-02 10:38:00 -0500 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2022-03-02 10:38:00 -0500 |
| commit | f861ffa660dc47c4017c220b94d10a64439d46a7 (patch) | |
| tree | 7ba56091d7713bf04e940afa9a2f8adcaeeb2a16 /tools/patman/checkpatch.py | |
| parent | f9a719e2954473f9be1f8c14a28288f943a00dd2 (diff) | |
| parent | 642e51addf918e0dd1b901efaa9f5706c12365b7 (diff) | |
Merge branch '2022-03-02-enable-pylint-in-CI' into next
To quote the author:
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
Diffstat (limited to 'tools/patman/checkpatch.py')
| -rw-r--r-- | tools/patman/checkpatch.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py index dd792efee0b..70ba561c268 100644 --- a/tools/patman/checkpatch.py +++ b/tools/patman/checkpatch.py @@ -125,7 +125,7 @@ def check_patch_parse(checkpatch_output, verbose=False): Returns: namedtuple containing: ok: False=failure, True=ok - problems: List of problems, each a dict: + problems (list of problems): each a dict: 'type'; error or warning 'msg': text message 'file' : filename @@ -252,6 +252,8 @@ def check_patches(verbose, args): if (len(result.problems) != result.errors + result.warnings + result.checks): print("Internal error: some problems lost") + # Python seems to get confused by this + # pylint: disable=E1133 for item in result.problems: sys.stderr.write( get_warning_msg(col, item.get('type', '<unknown>'), |