Stop ignoring E123 and E125 pycodestyle rules

Remove options from flake8 section in tox.ini

This change is one in the series of changes to
align os-brick with Cinder rules.

- E123 'closing bracket does not match indentation of
        opening bracket’s line'
  This option is ignored by default as it is not PEP8 enforced

- E125 'continuation line with same indent as next logical line'

See following for more information:
 https://github.com/PyCQA/pycodestyle

Change-Id: I439e677e66d59a48d4b2ac87eb0256146ff507e0
Signed-off-by: Maciej Kucia <maciej@kucia.net>
This commit is contained in:
Maciej Kucia 2017-05-21 23:01:55 +02:00
parent 4e175333e9
commit 32b4b24bfc
3 changed files with 5 additions and 6 deletions

View File

@ -106,7 +106,7 @@ class ScaleIOConnector(base.BaseLinuxConnector):
msg = (
_("ScaleIO volume %(volume_id)s not found at "
"expected path.") % {'volume_id': self.volume_id}
)
)
LOG.debug(msg)
raise exception.BrickException(message=msg)

View File

@ -26,13 +26,13 @@ class DISCOConnectorTestCase(test_connector.ConnectorTestCase):
volume = {
'name': 'a-disco-volume',
'disco_id': '1234567'
}
}
# Conf for test
conf = {
'ip': test_connector.MY_IP,
'port': 9898
}
}
def setUp(self):
super(DISCOConnectorTestCase, self).setUp()
@ -43,7 +43,7 @@ class DISCOConnectorTestCase(test_connector.ConnectorTestCase):
'conf': {
'server_ip': self.conf['ip'],
'server_port': self.conf['port']}
}
}
self.fake_volume_status = {'attached': True,
'detached': False}

View File

@ -59,7 +59,6 @@ commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasen
[flake8]
# H803 skipped on purpose per list discussion.
# E123, E125 skipped as they are invalid PEP-8.
# Following checks are ignored on purpose.
#
# E251 unexpected spaces around keyword / parameter equals
@ -82,7 +81,7 @@ commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasen
show-source = True
ignore = E123,E125,E251,E265,H405,H904
ignore = E251,E265,H405,H904
builtins = _
exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,build
max-complexity=30