From 01ec30cda363a179fad7fb0d9b91e771709c5c1d Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Tue, 12 Aug 2014 18:48:31 +0200 Subject: [PATCH] run_checks: Add main block --- scripts/run_checks.py | 47 ++++++++++++++++++++++++------------------- 1 file changed, 26 insertions(+), 21 deletions(-) diff --git a/scripts/run_checks.py b/scripts/run_checks.py index 548f12f81..6a76a07a3 100755 --- a/scripts/run_checks.py +++ b/scripts/run_checks.py @@ -221,27 +221,32 @@ def _get_args(checker): return args -argv = sys.argv[:] -check_unittest() -check_git() -for trg in CONFIG.get('DEFAULT', 'targets').split(','): - print("==================== {} ====================".format(trg)) - check_pep257(trg, _get_args('pep257')) - for chk in ('pylint', 'flake8'): - # FIXME what the hell is the flake8 exit status? - run(chk, trg, _get_args(chk)) - check_vcs_conflict(trg) +def main(): + argv = sys.argv[:] + check_unittest() + check_git() + for trg in CONFIG.get('DEFAULT', 'targets').split(','): + print("==================== {} ====================".format(trg)) + check_pep257(trg, _get_args('pep257')) + for chk in ('pylint', 'flake8'): + # FIXME what the hell is the flake8 exit status? + run(chk, trg, _get_args(chk)) + check_vcs_conflict(trg) -if '--setup' in argv: - print("==================== Setup checks ====================") - for chk in ('pyroma', 'check-manifest'): - run(chk, args=_get_args(chk)) + if '--setup' in argv: + print("==================== Setup checks ====================") + for chk in ('pyroma', 'check-manifest'): + run(chk, args=_get_args(chk)) -print("Exit status values:") -for (k, v) in status.items(): - print(' {} - {}'.format(k, v)) + print("Exit status values:") + for (k, v) in status.items(): + print(' {} - {}'.format(k, v)) -if all(val in (True, 0) for val in status): - sys.exit(0) -else: - sys.exit(1) + if all(val in (True, 0) for val in status): + return 0 + else: + return 1 + + +if __name__ == '__main__': + sys.exit(main())