diff options
author | Jeremy Bettis <jbettis@google.com> | 2022-08-16 13:05:23 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-08-17 22:09:35 +0000 |
commit | 20e14763fd8ea76156f2f064c23660f882eae8e7 (patch) | |
tree | 1fcff82ecff6692eae55cf4b6c5913c227a0e352 /pylintrc | |
parent | 5d79e92cba579a539d3e8b4dc2ed5be7696a4ea6 (diff) | |
download | chrome-ec-20e14763fd8ea76156f2f064c23660f882eae8e7.tar.gz |
ec: Adjust pylint disabled messages
Add several too-many-* disables to the pylintrc file. The very idea that
a function can have X of Ys but no more is not helpful.
Remove the now duplicated disables from the code.
Also, since pylint (at least as called from cros lint) doesn't inherit
settings from parent dirs, copy the disables to all child .pylintrc
files also.
BRANCH=None
BUG=None
TEST=git ls-files | grep 'py$' | xargs cros lint -v |& less
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: I9bb14cd09a066ddd6e2d45cd541cef6695ad36f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3833920
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'pylintrc')
-rw-r--r-- | pylintrc | 5 |
1 files changed, 4 insertions, 1 deletions
@@ -11,7 +11,10 @@ disable= bad-whitespace, # These have nothing to do with black, they are just annoying fixme, - too-many-arguments + too-many-arguments, + too-many-statements, + too-many-branches, + too-many-locals [format] |