openstack
/
heat
Code Issues Proposed changes

Merge "pep8: Permit ".)" at the end of resource docs"

This commit is contained in:
Zuul 2019-12-19 07:30:13 +00:00 committed by Gerrit Code Review
parent 1535ccf654 36a3eaaa35
commit cfc63544c5
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tools/custom_guidelines.py
View File

@ -199,7 +199,7 @@ class HeatCustomGuidelines(object):
'with uppercase letter') % error_key.title(),
'snippet': description})
self.print_guideline_error(**error_kwargs)
if not description.endswith('.'):
if not (description.endswith('.') or description.endswith('.)')):
error_kwargs.update(
{'message': _('%s description summary omitted '
'terminator at the end') % error_key.title(),
@ -250,7 +250,8 @@ class HeatCustomGuidelines(object):
if re.search("^(:param|:type|:returns|:rtype|:raises)",
line):
params = True
if not params and not doclines[-2].endswith('.'):
if not params and not (doclines[-2].endswith('.') or
doclines[-2].endswith('.)')):
error_kwargs.update(
{'message': _('%s description omitted '
'terminator at the end') % error_key.title(),