From ef83374fd340762fa983e9eefd2a5c9a4f22ed41 Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Sun, 26 Oct 2014 22:09:06 +0100 Subject: [PATCH] Clean up run_checks a bit. --- scripts/run_checks.py | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/scripts/run_checks.py b/scripts/run_checks.py index d38034640..b44cad822 100755 --- a/scripts/run_checks.py +++ b/scripts/run_checks.py @@ -276,22 +276,26 @@ def _checker_enabled(args, group, name): return name in args.checkers.split(',') +def _parse_args(): + """Parse commandline args via argparse.""" + parser = argparse.ArgumentParser(description='Run various checkers.') + parser.add_argument('-s', '--setup', help="Run additional setup checks", + action='store_true') + parser.add_argument('checkers', help="Checkers to run (or 'all')", + default='all', nargs='?') + return parser.parse_args() + + def main(): """Main entry point.""" - global config utils.change_cwd() read_files = config.read('.run_checks') if not read_files: raise IOError("Could not read config!") exit_status = collections.OrderedDict() exit_status_bool = {} - parser = argparse.ArgumentParser(description='Run various checkers.') - parser.add_argument('-s', '--setup', help="Run additional setup checks", - action='store_true') - parser.add_argument('checkers', help="Checkers to run (or 'all')", - default='all', nargs='?') - args = parser.parse_args() + args = _parse_args() checkers = _get_checkers() groups = ['global']