Fix Raises and update for pylint too-many-positional-arguments

This is in preview as ruff PLR0917, but we can't switch to that until it
is released generally.

Author: Qalthos

src/ansible_dev_environment/output.py

@@ -212,7 +212,7 @@ def to_lines(
 class Output:
     """Output functionality."""
 
-    def __init__(
+    def __init__(  # pylint: disable=too-many-positional-arguments
         self: Output,
         log_file: str,
         log_level: str,

src/ansible_dev_environment/subcommands/installer.py

@@ -151,7 +151,7 @@ def _install_galaxy_collections(
             collections: The collection objects.
 
         Raises:
-            SystemExit: If the collection installation fails.
+            SystemError: If the collection installation fails.
         """
         collections_str = " ".join(
             [f"'{collection.original}'" for collection in collections],
@@ -373,7 +373,7 @@ def _install_local_collection(
 
         Raises:
             RuntimeError: If tarball is not found or if more than one tarball is found.
-            SystemExit: If the collection installation fails.
+            SystemError: If the collection installation fails.
         """
         msg = f"Installing local collection from: {collection.build_dir}"
         self._output.info(msg)

src/ansible_dev_environment/tree.py

@@ -107,7 +107,7 @@ def is_scalar(obj: JSONVal) -> bool:
         """
         return isinstance(obj, str | int | float | bool) or obj is None
 
-    def _print_tree(  # noqa: C901, PLR0912
+    def _print_tree(  # noqa: C901, PLR0912  # pylint: disable=too-many-positional-arguments
         self: Tree,
         obj: JSONVal,
         is_last: bool,  # noqa: FBT001

src/ansible_dev_environment/utils.py

@@ -112,7 +112,7 @@ class Ansi:
     GREY = "\x1b[90m"
 
 
-def subprocess_run(  # noqa: PLR0913
+def subprocess_run(  # noqa: PLR0913  # pylint: disable=too-many-positional-arguments
     command: str,
     verbose: int,
     msg: str,

tests/unit/test_installer.py

@@ -725,7 +725,7 @@ def test_collection_pre_install(
 
 @pytest.mark.parametrize("first", (True, False), ids=["editable", "not_editable"])
 @pytest.mark.parametrize("second", (True, False), ids=["editable", "not_editable"])
-def test_reinstall_local_collection(  # noqa: PLR0913
+def test_reinstall_local_collection(  # noqa: PLR0913  # pylint: disable=too-many-positional-arguments
     first: bool,  # noqa: FBT001
     second: bool,  # noqa: FBT001
     tmp_path: Path,