From 6f38ea438ab03ff007dc239fe64752e8516f669b Mon Sep 17 00:00:00 2001 From: David Baumgold Date: Wed, 19 Mar 2014 14:21:54 -0400 Subject: [PATCH] Change `if not args.skip_collect` to `if args.collect` Simpler and clearer --- pavelib/assets.py | 22 +++++++++++++++++----- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/pavelib/assets.py b/pavelib/assets.py index 3f1064e500..62f629229c 100644 --- a/pavelib/assets.py +++ b/pavelib/assets.py @@ -88,10 +88,22 @@ def update_assets(args): Compile CoffeeScript and Sass, then collect static assets. """ parser = argparse.ArgumentParser(prog='paver update_assets') - parser.add_argument('system', type=str, nargs='*', default=['lms', 'studio'], help="lms or studio") - parser.add_argument('--settings', type=str, default="dev", help="Django settings module") - parser.add_argument('--debug', action='store_true', default=False, help="Disable Sass compression") - parser.add_argument('--skip-collect', action='store_true', default=False, help="Skip collection of static assets") + parser.add_argument( + 'system', type=str, nargs='*', default=['lms', 'studio'], + help="lms or studio", + ) + parser.add_argument( + '--settings', type=str, default="dev", + help="Django settings module", + ) + parser.add_argument( + '--debug', action='store_true', default=False, + help="Disable Sass compression", + ) + parser.add_argument( + '--skip-collect', dest='collect', action='store_false', default=True, + help="Skip collection of static assets", + ) args = parser.parse_args(args) compile_templated_sass(args.system, args.settings) @@ -99,5 +111,5 @@ def update_assets(args): compile_coffeescript() compile_sass(args.debug) - if not args.skip_collect: + if args.collect: collect_assets(args.system, args.settings)