From d0c115ed5778bcc0b0b1506e3c82a2e4b3bbaddf Mon Sep 17 00:00:00 2001 From: Omkar Pakki Date: Tue, 9 Jul 2019 22:28:48 +0530 Subject: [PATCH] Flake8 validation done --- fabfile/lib.py | 2 +- fabfile/tasks.py | 19 ++++++++++--------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/fabfile/lib.py b/fabfile/lib.py index 46430938..bafa1b4d 100644 --- a/fabfile/lib.py +++ b/fabfile/lib.py @@ -17,7 +17,7 @@ from fabvenv import virtualenv FABRIC_ROOT = os.path.dirname(__file__) project_root = os.path.dirname(FABRIC_ROOT) VENV_ROOT = os.path.expanduser(os.path.join('~', '.virtualenvs', 'pybitmessage-devops')) -PYTHONPATH = os.path.join(project_root, 'src', ) +PYTHONPATH = os.path.join(project_root, 'src',) def coerce_list(value): diff --git a/fabfile/tasks.py b/fabfile/tasks.py index a5d8d972..fca47d85 100644 --- a/fabfile/tasks.py +++ b/fabfile/tasks.py @@ -32,13 +32,14 @@ def get_tool_results(file_list): 'pycodestyle_violations': get_filtered_pycodestyle_output(path_to_file), 'flake8_violations': get_filtered_flake8_output(path_to_file), 'pylint_violations': get_filtered_pylint_output(path_to_file), - 'path_to_file': path_to_file, - 'total_violations': sum([ - len(result['pycodestyle_violations']), - len(result['flake8_violations']), - len(result['pylint_violations']), - ]) + 'path_to_file': path_to_file } + result['total_violations'] = sum([ + len(result['pycodestyle_violations']), + len(result['flake8_violations']), + len(result['pylint_violations']) + ]) + results.append(result) return results @@ -73,17 +74,17 @@ def print_results(results, top, verbose, details): print("pycodestyle:") for detail in flatten(item['pycodestyle_violations']): print(detail) - print + print() print("flake8:") for detail in flatten(item['flake8_violations']): print(detail) - print + print() print("pylint:") for detail in flatten(item['pylint_violations']): print(detail) - print + print() def sort_and_slice(results, top):