235cbb839e
pylint 2.2.2 which we currently use does not support python 3.8 [1]. Let's use a newer version of pylint. pylint now declares strict versions for astroid and isort, so we do not need to manage them in our test-requirements.txt. All new checks are disabled temporarily. Let's visit them later and judge whether we disable them permanently one by one. [1] https://github.com/PyCQA/pylint/issues/2737 Change-Id: I106b250e632305365aaab4d881c9ba5ea93505b3
112 lines
2.9 KiB
INI
112 lines
2.9 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=test,tests,tests.py,local_settings.py,horizon_wsgi.py
|
|
|
|
[Messages Control]
|
|
disable=
|
|
# "F" Fatal errors that prevent further processing
|
|
# "I" Informational noise
|
|
locally-disabled,
|
|
# "E" Error for important programming issues (likely bugs)
|
|
method-hidden,
|
|
no-member,
|
|
no-value-for-parameter, # TODO
|
|
not-callable,
|
|
# "W" Warnings for stylistic problems or minor programming issues
|
|
arguments-differ,
|
|
attribute-defined-outside-init,
|
|
bad-indentation,
|
|
broad-except,
|
|
deprecated-method, # TODO
|
|
fixme,
|
|
keyword-arg-before-vararg, # TODO
|
|
pointless-string-statement,
|
|
protected-access,
|
|
raise-missing-from, # TODO
|
|
redefined-builtin,
|
|
redefined-outer-name,
|
|
signature-differs,
|
|
super-init-not-called,
|
|
unidiomatic-typecheck,
|
|
unused-argument,
|
|
unused-wildcard-import,
|
|
useless-else-on-loop,
|
|
using-constant-test, # TODO
|
|
wildcard-import,
|
|
# "C" Coding convention violations
|
|
abstract-method,
|
|
anomalous-backslash-in-string,
|
|
bad-builtin,
|
|
bad-continuation,
|
|
deprecated-lambda,
|
|
global-statement,
|
|
import-outside-toplevel, # TODO
|
|
invalid-name,
|
|
missing-docstring,
|
|
superfluous-parens,
|
|
too-many-lines,
|
|
unused-variable,
|
|
wrong-import-order, # TODO
|
|
# "R" Refactor recommendations
|
|
duplicate-code,
|
|
inconsistent-return-statements, # TODO
|
|
interface-not-implemented,
|
|
no-else-continue, # TODO
|
|
no-else-raise, # TODO
|
|
no-else-return,
|
|
no-self-use,
|
|
super-with-arguments, # TODO
|
|
too-many-ancestors,
|
|
too-many-arguments,
|
|
too-many-branches,
|
|
too-many-function-args,
|
|
too-many-instance-attributes,
|
|
too-many-locals,
|
|
too-many-return-statements,
|
|
too-many-statements,
|
|
unnecessary-comprehension, # TODO
|
|
useless-object-inheritance
|
|
|
|
[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 lowecased with underscores
|
|
method-rgx=([a-z_][a-z0-9_]{2,50}|setUp|tearDown)$
|
|
|
|
# Module names matching keystone-* are ok (files in bin/)
|
|
module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+)|(keystone-[a-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=80
|
|
|
|
[Design]
|
|
max-public-methods=100
|
|
min-public-methods=0
|
|
max-args=6
|
|
|
|
[Variables]
|
|
|
|
# List of additional names supposed to be defined in builtins. Remember that
|
|
# you should avoid to define new builtins when possible.
|
|
# _ is used by our localization
|
|
additional-builtins=_
|
|
|
|
[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
|