Fix to add_ballot_set.py arguments

TabbycatDebate · Aug 4, 2015 · 6b68df0 · 6b68df0
1 parent 642307b
commit 6b68df0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/data/utils/add_ballot_set.py b/data/utils/add_ballot_set.py
@@ -82,7 +82,7 @@ def do():
     parser.add_argument("debate", type=int, nargs='+', help="Debate ID(s) to add to")
     parser.add_argument("-t", "--type", type=str, help="'tabroom' or 'public'", choices=SUBMITTER_TYPE_MAP.keys(), default="tabroom")
     parser.add_argument("-u", "--user", type=str, help="User ID", default="original")
-    status = parser.add_mutually_exclusive_group(required=True)
+    status = parser.add_mutually_exclusive_group()
     status.add_argument("-d", "--discarded", action="store_true", help="Ballot set is discarded")
     status.add_argument("-c", "--confirmed", action="store_true", help="Ballot set is confirmed")
     parser.add_argument("-m", "--min-score", type=float, help="Minimum speaker score (for substantive)", default=72)

diff --git a/deploy_heroku.py b/deploy_heroku.py
@@ -22,6 +22,8 @@
     help="Heroku Postgres plan (default hobby-dev)")
 parser.add_argument("--import-tournament", type=str, metavar="IMPORT_DIR",
     help="Also run the import_tournament command, importing from IMPORT_DIR")
+parser.add_argument("--dry-run", action="store_true", default=False,
+    help="Print commands, don't run them.")
 
 config_group = parser.add_argument_group("heroku configuration settings")
 config_group.add_argument("--public-cache-timeout", type=int, default=None, metavar="TIMEOUT",
@@ -60,7 +62,8 @@ def print_command(command):
 
 def run_command(command):
     print_command(command)
-    subprocess.check_call(command, **subprocess_kwargs)
+    if not args.dry_run:
+        subprocess.check_call(command, **subprocess_kwargs)
 
 def run_heroku_command(command):
     command.insert(0, "heroku")