diff options
| author |   Vratko Polak <vrpolak@cisco.com> | 2019-12-04 13:24:07 +0100 |
|---|---|---|
| committer |   Peter Mikus <pmikus@cisco.com> | 2019-12-05 07:03:57 +0000 |
| commit | 063abf35e81deaf749ebbcfee339fbd1d9e89412 (patch) | |
| tree | c932eac04f162c46b0f3c38db10105b945a617cb /pylint.cfg | |
| parent | 6221f1b96d2a167c6db74ff26cd7ec7906ae9486 (diff) | |
Deal with some "pylint: disable=" comments
+ When possible, fix the violation.
+ Else, add a comment:
+ An explanation (if not already present) and keep disable.
+ A TODO (if not already present) and remove the disable.
- This makes tox job report more pylint violations,
but any such violation is fixable and should be fixed.
- Although some need to be fixed in VPP, such as enum item long names.
Change-Id: I48604b5eda070083d79dff1439620dbd9e798e1f
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Diffstat (limited to 'pylint.cfg')
| -rw-r--r-- | pylint.cfg | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/pylint.cfg b/pylint.cfg index 5ae4ed551b..82b3a06215 100644 --- a/pylint.cfg +++ b/pylint.cfg @@ -42,8 +42,9 @@ extension-pkg-whitelist=numpy, scipy # --enable=similarities". If you want to run only the classes checker, but have # no Warning level messages displayed, use"--disable=all --enable=classes # --disable=W" -disable=redefined-variable-type, locally-disabled, locally-enabled +#disable=redefined-variable-type, locally-disabled, locally-enabled +# TODO: Add explanation when disabling an id, either locally or globally. [REPORTS] |