From 063abf35e81deaf749ebbcfee339fbd1d9e89412 Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Wed, 4 Dec 2019 13:24:07 +0100 Subject: 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 --- pylint.cfg | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'pylint.cfg') 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] -- cgit 1.2.3-korg