diff --git a/.pylintrc b/.pylintrc index cb448fb0d50..2cbdd38d499 100644 --- a/.pylintrc +++ b/.pylintrc @@ -38,12 +38,14 @@ disable= global-variable-not-assigned, no-init, non-parent-init-called, + not-callable, protected-access, redefined-builtin, redefined-outer-name, signature-differs, star-args, super-init-not-called, + super-on-old-class, unpacking-non-sequence, unused-argument, unused-import, diff --git a/tox.ini b/tox.ini index dc8a5503d5e..54d91612310 100644 --- a/tox.ini +++ b/tox.ini @@ -50,11 +50,15 @@ sitepackages = True downloadcache = ~/cache/pip [testenv:pep8] -commands = +deps = + {[testenv]deps} + pylint +commands= # If it is easier to add a check via a shell script, consider adding it in this file sh ./tools/misc-sanity-checks.sh flake8 neutron-db-manage check_migration + pylint --rcfile=.pylintrc --output-format=colorized {posargs:neutron} whitelist_externals = sh [testenv:cover] @@ -81,13 +85,6 @@ show-source = true builtins = _ exclude = ./.*,build,dist,neutron/openstack/common/* -[testenv:pylint] -deps = - {[testenv]deps} - pylint -commands = - pylint --rcfile=.pylintrc --output-format=colorized {posargs:neutron} - [hacking] import_exceptions = neutron.i18n local-check-factory = neutron.hacking.checks.factory