Correct failures for check H703
The H703 "Multiple positional placeholders" rule was failing and ignored. Now it's enforced. Change-Id: Id6d3bc1767e264d3f07c4e14758d67e10223f02f
This commit is contained in:
parent
c252a8095d
commit
a31ed5ca4f
|
@ -1220,8 +1220,8 @@ class AuthProtocol(object):
|
|||
current_mode = stat.S_IMODE(os.stat(self._signing_dirname).st_mode)
|
||||
if current_mode != stat.S_IRWXU:
|
||||
self._LOG.warning(
|
||||
_LW('signing_dir mode is %s instead of %s'),
|
||||
oct(current_mode), oct(stat.S_IRWXU))
|
||||
_LW('signing_dir mode is %(mode)s instead of %(need)s'),
|
||||
{'mode': oct(current_mode), 'need': oct(stat.S_IRWXU)})
|
||||
else:
|
||||
os.makedirs(self._signing_dirname, stat.S_IRWXU)
|
||||
|
||||
|
|
3
tox.ini
3
tox.ini
|
@ -36,8 +36,7 @@ commands = oslo_debug_helper {posargs}
|
|||
# F821: undefined name
|
||||
# H304: no relative imports
|
||||
# H405: multi line docstring summary not separated with an empty line
|
||||
# H703: Multiple positional placeholders
|
||||
ignore = E122,F821,H304,H405,H703
|
||||
ignore = E122,F821,H304,H405
|
||||
show-source = True
|
||||
exclude = .venv,.tox,dist,doc,*egg,build,*openstack/common*
|
||||
|
||||
|
|
Loading…
Reference in New Issue