summaryrefslogtreecommitdiff
path: root/.pylintrc
diff options
context:
space:
mode:
authorAkihiro Motoki <amotoki@gmail.com>2017-06-07 13:01:03 +0000
committergaryk <gkotton@vmware.com>2017-06-08 09:49:46 +0000
commit17ed89cbc12db9d9c8882f1e4162547725d030ec (patch)
tree358df29428c264429b57f02bed212c9b450f0a2f /.pylintrc
parent1e98b0a83eec8eaac3b1bf94f47614ee6b3cb507 (diff)
downloadneutron-17ed89cbc12db9d9c8882f1e4162547725d030ec.tar.gz
Update pylint disable list to pass pylint 1.7.1 checks
Recenly pylint version was bumped to 1.7.1 in global-requirements.txt. Pylint 1.7.1 adds more checks and breaks the neutron pep8 job. This commit adds new checks to the disabled list to avoid the gate breakage. Individual disabled tests can be fixed later. The new disabled list is compatible with pylint 1.4.5, so we can merge this regardless of pylint bump is reverted or not. Closes-Bug: #1696403 Change-Id: If6228d0626413049b82f9d75bcdf3bea7ddacde5
Diffstat (limited to '.pylintrc')
-rw-r--r--.pylintrc16
1 files changed, 16 insertions, 0 deletions
diff --git a/.pylintrc b/.pylintrc
index 8a0230ab96..1c59579901 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -18,6 +18,7 @@ disable=
no-member,
no-method-argument,
no-self-argument,
+ not-an-iterable,
# "W" Warnings for stylistic problems or minor programming issues
abstract-method,
arguments-differ,
@@ -30,6 +31,7 @@ disable=
expression-not-assigned,
fixme,
global-statement,
+ literal-comparison,
no-init,
non-parent-init-called,
not-callable,
@@ -44,19 +46,32 @@ disable=
unused-argument,
unused-import,
unused-variable,
+ useless-super-delegation,
# TODO(dougwig) - disable nonstandard-exception while we have neutron_lib shims
nonstandard-exception,
# "C" Coding convention violations
bad-continuation,
+ consider-iterating-dictionary,
+ consider-using-enumerate,
invalid-name,
+ len-as-condition,
+ misplaced-comparison-constant,
missing-docstring,
+ singleton-comparison,
superfluous-parens,
+ ungrouped-imports,
+ wrong-import-order,
# "R" Refactor recommendations
abstract-class-little-used,
abstract-class-not-used,
+ consider-merging-isinstance,
+ consider-using-ternary,
duplicate-code,
interface-not-implemented,
+ no-else-return,
no-self-use,
+ redefined-argument-from-local,
+ simplifiable-if-statement,
too-few-public-methods,
too-many-ancestors,
too-many-arguments,
@@ -64,6 +79,7 @@ disable=
too-many-instance-attributes,
too-many-lines,
too-many-locals,
+ too-many-nested-blocks,
too-many-public-methods,
too-many-return-statements,
too-many-statements