Selectively revert bdd0ce4

The trailing comma check is useful for iterables but not calls. Remove
the trailing commas from calls and turn off the warning.

Signed-off-by: Nathan Chancellor <nathan@kernel.org>

Author: nathanchance

boot-qemu.py

@@ -75,28 +75,28 @@ def parse_arguments():
         "--shell",
         action="store_true",
         help=
-        "Instead of immediately shutting down the machine upon successful boot, pass 'rdinit=/bin/sh' on the kernel command line to allow interacting with the machine via a shell.",
+        "Instead of immediately shutting down the machine upon successful boot, pass 'rdinit=/bin/sh' on the kernel command line to allow interacting with the machine via a shell."
     )
     parser.add_argument(
         "-k",
         "--kernel-location",
         required=True,
         type=str,
         help=
-        "Path to kernel image or kernel build folder to search for image in. Can be an absolute or relative path.",
+        "Path to kernel image or kernel build folder to search for image in. Can be an absolute or relative path."
     )
     parser.add_argument(
         "--no-kvm",
         action="store_true",
         help=
-        "Do not use KVM for acceleration even when supported (only recommended for debugging).",
+        "Do not use KVM for acceleration even when supported (only recommended for debugging)."
     )
     parser.add_argument(
         "-s",
         "--smp",
         type=int,
         help=
-        "Number of processors for virtual machine. By default, only machines spawned with KVM will use multiple vCPUS.",
+        "Number of processors for virtual machine. By default, only machines spawned with KVM will use multiple vCPUS."
     )
     parser.add_argument(
         "-t",
@@ -656,7 +656,7 @@ def get_qemu_args(cfg):
         dtb = kernel.parent.joinpath(dtb_dir, dtb)
         if not dtb.exists():
             utils.die(
-                f"'{dtb.stem}' is required for booting but it could not be found at '{dtb}'",
+                f"'{dtb.stem}' is required for booting but it could not be found at '{dtb}'"
             )
 
         qemu_args += ["-dtb", dtb]

boot-uml.py

@@ -24,15 +24,15 @@ def parse_arguments():
         "--interactive",
         action="store_true",
         help=
-        "Instead of immediately shutting down upon successful boot, pass 'init=/bin/sh' to the UML executable to allow interacting with UML via a shell.",
+        "Instead of immediately shutting down upon successful boot, pass 'init=/bin/sh' to the UML executable to allow interacting with UML via a shell."
     )
     parser.add_argument(
         "-k",
         "--kernel-location",
         required=True,
         type=str,
         help=
-        "Path to UML executable ('linux') or kernel build folder to search for executable in. Can be an absolute or relative path.",
+        "Path to UML executable ('linux') or kernel build folder to search for executable in. Can be an absolute or relative path."
     )
 
     return parser.parse_args()

ruff.toml

@@ -4,7 +4,6 @@ select = [
     'ARG', # flake8-unused-arguments
     'B', # flake8-bugbear
     'C4', # flake8-comprehensions
-    'COM', # flake8-commas
     'E', # pycodestyle
     'F', # pyflakes
     'PIE', # flake8-pie

utils.py

@@ -14,9 +14,8 @@ def check_cmd(cmd):
         cmd (str): External command name or path.
     """
     if not shutil.which(cmd):
-        die(
-            f"The external command '{cmd}' is needed but it could not be found in PATH, please install it!",
-        )
+        die(f"The external command '{cmd}' is needed but it could not be found in PATH, please install it!"
+            )
 
 
 def die(string):
@@ -58,9 +57,8 @@ def get_full_kernel_path(kernel_location, image, arch=None):
         # Otherwise, it is in the architecture's boot directory
         else:
             if not arch:
-                die(
-                    f"Kernel image ('{image}') is in the arch/ directory but 'arch' was not provided!",
-                )
+                die(f"Kernel image ('{image}') is in the arch/ directory but 'arch' was not provided!"
+                    )
             kernel = kernel_location.joinpath("arch", arch, "boot", image)
 
     if not kernel.exists():