Run flake8 also on cinder/*/openstack

exclude=openstack also matches cinder/api/openstack,
which is however not merged from oslo-incubator.
Exclude=common instead, and explicitely include
cinder/common

Change-Id: I331d8c0d17e519386ae72caeca8a336eecde3ad3
This commit is contained in:
Dirk Mueller 2013-06-14 23:37:11 +02:00
parent 4c50624622
commit c1da47d7ce
2 changed files with 4 additions and 5 deletions

View File

@ -183,12 +183,11 @@ class XMLDeserializer(TextDeserializer):
"""Search a nodes children for the first child with a given name."""
for node in parent.childNodes:
if (node.localName == name and
node.namespaceURI and
node.namespaceURI == namespace):
node.namespaceURI and
node.namespaceURI == namespace):
return node
return None
def find_first_child_named(self, parent, name):
"""Search a nodes children for the first child with a given name"""
for node in parent.childNodes:

View File

@ -21,7 +21,7 @@ downloadcache = ~/cache/pip
[testenv:pep8]
sitepackages = False
commands =
flake8 {posargs}
flake8 {posargs} . cinder/common
flake8 --filename=cinder* bin
[testenv:pylint]
@ -45,4 +45,4 @@ commands = {posargs}
[flake8]
ignore = E711,E712,F401,F403,F811,F841,H302,H303,H304,H402,H404
builtins = _
exclude = .venv,.tox,dist,doc,openstack,*egg,build
exclude = .git,.venv,.tox,dist,doc,common,*egg,build