diff --git a/stackalytics/processor/driverlog.py b/stackalytics/processor/driverlog.py index e5459ff55..b58c61ede 100644 --- a/stackalytics/processor/driverlog.py +++ b/stackalytics/processor/driverlog.py @@ -22,9 +22,7 @@ LOG = logging.getLogger(__name__) def _find_vote(review, ci_id, patch_set_number): - """ - Finds vote corresponding to ci_id - """ + """Finds vote corresponding to ci_id.""" for patch_set in review['patchSets']: if patch_set['number'] == patch_set_number: for approval in (patch_set.get('approvals') or []): @@ -38,9 +36,7 @@ def _find_vote(review, ci_id, patch_set_number): def find_ci_result(review, ci_map): - """ - For a given stream of reviews yields results produced by CIs - """ + """For a given stream of reviews yields results produced by CIs.""" review_id = review['id'] review_number = review['number'] diff --git a/stackalytics/processor/record_processor.py b/stackalytics/processor/record_processor.py index be180864d..1e9065e28 100644 --- a/stackalytics/processor/record_processor.py +++ b/stackalytics/processor/record_processor.py @@ -326,8 +326,9 @@ class RecordProcessor(object): return mark def _process_review(self, record): - """ - Process a review. Review spawns into records of three types: + """Process a review. + + Review spawns into records of three types: * review - records that a user created review request * patch - records that a user submitted another patch set * mark - records that a user set approval mark to given review diff --git a/tox.ini b/tox.ini index b285340d6..b4f652166 100644 --- a/tox.ini +++ b/tox.ini @@ -53,8 +53,7 @@ commands = python setup.py build_sphinx [flake8] # E125 continuation line does not distinguish itself from next logical line -# H404 multi line docstring should start with a summary -ignore = E125,H404 +ignore = E125 show-source = true builtins = _ exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,tools,build