do not require --minify with --minify-safe-only

thisismypassport · Jun 21, 2023 · a24a375 · a24a375
1 parent 84080e9
commit a24a375
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/run_bbs_tests.py b/run_bbs_tests.py
@@ -148,7 +148,7 @@ def process_output(kind, output):
         best_path_for_pico8 = compress_path
 
     if g_opts.all or g_opts.only_safe_minify:
-        safe_minify_results = run_code(uncompress_path, safe_minify_path, "--minify", "--minify-safe-only", "--count", "--parsable-count")
+        safe_minify_results = run_code(uncompress_path, safe_minify_path, "--minify-safe-only", "--count", "--parsable-count")
         process_output("safe_minify", check_run("%s:safe_minify" % cart, safe_minify_results, parse_meta=True))
         best_path_for_pico8 = safe_minify_path
 

diff --git a/run_tests.py b/run_tests.py
@@ -100,8 +100,7 @@ def run():
              "--no-minify-spaces", "--no-minify-lines", pico8_output="output.p8.printh")
     run_test("minrename", "input.p8", "output_minrename.p8", "--minify",
              "--preserve", "*,*.*", pico8_output="output.p8.printh")
-    run_test("auto_minrename", "input.p8", "output_minrename.p8", "--minify",
-             "--minify-safe-only")
+    run_test("auto_minrename", "input.p8", "output_minrename.p8", "--minify-safe-only")
     run_test("minifytokens", "input.p8", "output_tokens.p8", "--minify",
              "--no-minify-spaces", "--no-minify-lines", "--no-minify-comments", "--no-minify-rename")
              # pico8_output="output.p8.printh" - broken by comment bug in pico8 v0.2.5g...

diff --git a/shrinko8.py b/shrinko8.py
@@ -128,7 +128,7 @@ def main(raw_args):
             "undefined": not args.no_lint_undefined,
         }
 
-    if args.minify:
+    if args.minify or args.minify_safe_only:
         args.minify = {
             "lines": not args.no_minify_lines,
             "wspace": not args.no_minify_spaces,