ruff: Fix PLW1510 `subprocess.run` without explicit `check` argument.

Signed-off-by: Anders Kaseorg <anders@zulip.com>

puppet/zulip/files/nagios_plugins/zulip_postgresql_backups/check_postgresql_backup

@@ -41,6 +41,7 @@ skip_backups = subprocess.run(
     ["crudini", "--get", "/etc/zulip/zulip.conf", "postgresql", "skip_backups"],
     capture_output=True,
     text=True,
+    check=False,
 )
 if skip_backups.returncode == 0 and skip_backups.stdout.strip().lower() in [
     1,

scripts/lib/upgrade-zulip-stage-2

@@ -399,7 +399,9 @@ has_puppet_changes = True
 if not args.skip_puppet and IS_SERVER_UP:
     logging.info("Pre-checking for puppet changes...")
     try_puppet = subprocess.run(
-        ["./scripts/zulip-puppet-apply", "--noop", "--force"], stdout=subprocess.DEVNULL
+        ["./scripts/zulip-puppet-apply", "--noop", "--force"],
+        stdout=subprocess.DEVNULL,
+        check=False,
     )
     if try_puppet.returncode == 0:
         if minimal_change:

scripts/restart-server

@@ -185,6 +185,7 @@ if has_application_server():
         uwsgi_status = subprocess.run(
             ["supervisorctl", "status", "zulip-django"],
             stdout=subprocess.DEVNULL,
+            check=False,
         )
         if uwsgi_status.returncode == 0:
             logging.info("Starting rolling restart of django server")

tools/lib/pretty_print.py

@@ -180,7 +180,7 @@ Proposed {BOLDRED}diff{ENDC} for {CYAN}{fn}{ENDC}:
             """,
             flush=True,
         )
-        subprocess.run(["diff", fn, "-"], input=phtml, text=True)
+        subprocess.run(["diff", fn, "-"], input=phtml, text=True, check=False)
         print(
             f"""
 ---

zerver/tests/test_email_mirror.py

@@ -1493,6 +1493,7 @@ class TestScriptMTA(ZulipTestCase):
             input=mail,
             stdout=subprocess.PIPE,
             text=True,
+            check=False,
         )
         self.assertEqual(
             p.stdout,