openstack/monasca-persister
Code Issues Proposed changes

Merge "Fixes H201: no 'except:' at least use 'except Exception:'"

Browse Source
This commit is contained in:
Jenkins 2016-08-01 06:24:20 +00:00 committed by Gerrit Code Review
commit 346c684c03
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
monasca_persister/repositories/persister.py
View File

@ -73,7 +73,7 @@ class Persister(object):
if len(self._data_points) >= self._database_batch_size: if len(self._data_points) >= self._database_batch_size:
self._flush() self._flush()
except: except Exception:
LOG.exception( LOG.exception(
'Persister encountered fatal exception processing ' 'Persister encountered fatal exception processing '
'messages. ' 'messages. '

3
tox.ini
View File

@ -26,8 +26,7 @@ commands = {posargs}
[flake8] [flake8]
max-line-length = 120 max-line-length = 120
# TODO: ignored checks should be enabled in the future # TODO: ignored checks should be enabled in the future
# H201 no 'except:' at least use 'except Exception:'
# H405 multi line docstring summary not separated with an empty line # H405 multi line docstring summary not separated with an empty line
# H904 Wrap long lines in parentheses instead of a backslash # H904 Wrap long lines in parentheses instead of a backslash
ignore = F821,H201,H405,H904,E126,E125,H306,E302,E122 ignore = F821,H405,H904,E126,E125,H306,E302,E122
exclude=.venv,.git,.tox,dist,*openstack/common*,*egg,build exclude=.venv,.git,.tox,dist,*openstack/common*,*egg,build