Bachmann1234#378 Explicitly check for FileNotFound instead

d-perl · Dec 14, 2023 · 13ba02b · 13ba02b
1 parent 0410b2b
commit 13ba02b
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/diff_cover/command_runner.py b/diff_cover/command_runner.py
@@ -51,10 +51,13 @@ def run_command_for_code(command):
     """
     Returns command's exit code.
     """
-    process = subprocess.Popen(
-        command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, shell=True
-    )
-    process.communicate()
+    try:
+        process = subprocess.Popen(
+            command, stdout=subprocess.PIPE, stderr=subprocess.PIPE
+        )
+        process.communicate()
+    except FileNotFoundError:
+        return 1
     return process.returncode