ac3c95c1b4
The older version of pylint being used does not work correctly under python 3. In order to be able to update the pylint job to run under python 3, we need to update the tool. Aslo fix a couple of pylint issues: - Fix import order issues. - Fix 'no-else-return' checks. Added the following to the blacklist: - inconsistent-return-statements - catching-non-exception - using-constant-test Change-Id: I91a304241ebdb4d6bd15509cae8463d7b43b3c0f Signed-off-by: Chuck Short <chucks@redhat.com>
87 lines
2.3 KiB
INI
87 lines
2.3 KiB
INI
# The format of this file isn't really documented; just use --generate-rcfile
|
|
[MASTER]
|
|
# Add <file or directory> to the black list. It should be a base name, not a
|
|
# path. You may set this option multiple times.
|
|
ignore=.git,tests
|
|
|
|
[MESSAGES CONTROL]
|
|
# NOTE: This list is copied from neutron, the options which do not need to be
|
|
# suppressed have been removed.
|
|
disable=
|
|
# "F" Fatal errors that prevent further processing
|
|
# "I" Informational noise
|
|
# "E" Error for important programming issues (likely bugs)
|
|
no-member,
|
|
# "W" Warnings for stylistic problems or minor programming issues
|
|
arguments-differ,
|
|
attribute-defined-outside-init,
|
|
broad-except,
|
|
fixme,
|
|
protected-access,
|
|
redefined-outer-name,
|
|
unused-argument,
|
|
useless-super-delegation,
|
|
# "C" Coding convention violations
|
|
bad-continuation,
|
|
invalid-name,
|
|
missing-docstring,
|
|
# "R" Refactor recommendations
|
|
duplicate-code,
|
|
no-self-use,
|
|
too-few-public-methods,
|
|
too-many-ancestors,
|
|
too-many-arguments,
|
|
too-many-branches,
|
|
too-many-instance-attributes,
|
|
too-many-lines,
|
|
too-many-locals,
|
|
too-many-public-methods,
|
|
too-many-return-statements,
|
|
inconsistent-return-statements,
|
|
catching-non-exception,
|
|
using-constant-test,
|
|
too-many-statements
|
|
|
|
[BASIC]
|
|
# Variable names can be 1 to 31 characters long, with lowercase and underscores
|
|
variable-rgx=[a-z_][a-z0-9_]{0,30}$
|
|
|
|
# Argument names can be 2 to 31 characters long, with lowercase and underscores
|
|
argument-rgx=[a-z_][a-z0-9_]{1,30}$
|
|
|
|
# Method names should be at least 3 characters long
|
|
# and be lowercased with underscores
|
|
method-rgx=([a-z_][a-z0-9_]{2,}|setUp|tearDown)$
|
|
|
|
# Module names matching
|
|
module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$
|
|
|
|
# Don't require docstrings on tests.
|
|
no-docstring-rgx=((__.*__)|([tT]est.*)|setUp|tearDown)$
|
|
|
|
[FORMAT]
|
|
# Maximum number of characters on a single line.
|
|
max-line-length=79
|
|
|
|
[VARIABLES]
|
|
# List of additional names supposed to be defined in builtins. Remember that
|
|
# you should avoid to define new builtins when possible.
|
|
additional-builtins=
|
|
|
|
[CLASSES]
|
|
# List of interface methods to ignore, separated by a comma.
|
|
ignore-iface-methods=
|
|
|
|
[IMPORTS]
|
|
# Deprecated modules which should not be used, separated by a comma
|
|
deprecated-modules=
|
|
|
|
[TYPECHECK]
|
|
# List of module names for which member attributes should not be checked
|
|
ignored-modules=six.moves,_MovedItems
|
|
|
|
[REPORTS]
|
|
# Tells whether to display a full report or only the messages
|
|
reports=no
|
|
|