chore(ruff): fix PLW1510 subprocess.run without explicit check ar…

…gument
onekey-sec · Sep 20, 2023 · 083c5d3 · 083c5d3
1 parent e96f769
commit 083c5d3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/update-vendored-nix-dependencies.py b/.github/workflows/update-vendored-nix-dependencies.py
@@ -55,7 +55,8 @@ def create_pr(
 def update_dependencies():
     with NamedTemporaryFile() as log:
         subprocess.run(
-            ["nvfetcher", "--build-dir", "nix/_sources", "--changelog", log.name]
+            ["nvfetcher", "--build-dir", "nix/_sources", "--changelog", log.name],
+            check=True,
         )
         return Path(log.name).read_text()
 

diff --git a/unblob/extractors/command.py b/unblob/extractors/command.py
@@ -54,6 +54,7 @@ def no_op():
                 stdout=stdout_file,
                 stderr=subprocess.PIPE,
                 timeout=COMMAND_TIMEOUT,
+                check=False,
             )
             if res.returncode != 0:
                 error_report = ExtractCommandFailedReport(