Adds pylint section in tox.ini in oslo.config to check for unused imports
Fixes bug #1157597 Change-Id: I58057cdde6064dacf6689560ec6439fd6dc7cf4f
This commit is contained in:
parent
dadd3ea623
commit
5439d6bf29
34
tools/lint.py
Normal file
34
tools/lint.py
Normal file
@ -0,0 +1,34 @@
|
||||
# vim: tabstop=4 shiftwidth=4 softtabstop=4
|
||||
|
||||
# Copyright (c) 2013 Intel Corporation.
|
||||
# All Rights Reserved.
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
||||
# not use this file except in compliance with the License. You may obtain
|
||||
# a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
||||
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||
# License for the specific language governing permissions and limitations
|
||||
# under the License.
|
||||
#
|
||||
# @author: Zhongyue Luo, Intel Corporation.
|
||||
#
|
||||
|
||||
import sys
|
||||
|
||||
from pylint import lint
|
||||
|
||||
|
||||
ENABLED_PYLINT_MSGS = ['W0611']
|
||||
|
||||
|
||||
def main(dirpath):
|
||||
enable_opt = '--enable=%s' % ','.join(ENABLED_PYLINT_MSGS)
|
||||
lint.Run(['--reports=n', '--disable=all', enable_opt, dirpath])
|
||||
|
||||
if __name__ == '__main__':
|
||||
main(sys.argv[1])
|
7
tox.ini
7
tox.ini
@ -1,5 +1,5 @@
|
||||
[tox]
|
||||
envlist = py26,py27,pep8
|
||||
envlist = py26,py27,pep8,pylint
|
||||
|
||||
[testenv]
|
||||
setenv = VIRTUAL_ENV={envdir}
|
||||
@ -11,6 +11,11 @@ commands = nosetests --with-doctest --exclude-dir=tests/testmods {posargs}
|
||||
deps = pep8==1.3.3
|
||||
commands = pep8 --repeat --show-source --exclude=.tox,dist,doc,*.egg --ignore=E125 .
|
||||
|
||||
[testenv:pylint]
|
||||
deps = pylint>=0.26.0
|
||||
commands = python ./tools/lint.py ./oslo
|
||||
python ./tools/lint.py ./tests
|
||||
|
||||
[testenv:cover]
|
||||
setenv = NOSE_WITH_COVERAGE=1
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user