chore: pre-commit autoupdate (#256)

* chore: pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/tox-dev/tox-ini-fmt: 1.3.1 → 1.4.1](tox-dev/tox-ini-fmt@1.3.1...1.4.1)
- [github.com/astral-sh/ruff-pre-commit: v0.6.3 → v0.6.9](astral-sh/ruff-pre-commit@v0.6.3...v0.6.9)
- [github.com/jsh9/pydoclint: 0.5.6 → 0.5.9](jsh9/pydoclint@0.5.6...0.5.9)
- [github.com/pycqa/pylint.git: v3.2.7 → v3.3.1](https://github.com/pycqa/pylint.git/compare/v3.2.7...v3.3.1)

* 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.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kate Case <this.is@katherineca.se>

.pre-commit-config.yaml

@@ -7,7 +7,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: check-added-large-files
       - id: check-merge-conflict
@@ -54,12 +54,12 @@ repos:
       - id: toml-sort-fix
 
   - repo: https://github.com/tox-dev/tox-ini-fmt
-    rev: 1.3.1
+    rev: 1.4.1
     hooks:
       - id: tox-ini-fmt
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.6.3
+    rev: v0.6.9
     hooks:
       - id: ruff
         args:
@@ -72,15 +72,15 @@ repos:
         name: Spell check with cspell
 
   - repo: https://github.com/jsh9/pydoclint
-    rev: "0.5.6"
+    rev: "0.5.9"
     hooks:
       - id: pydoclint
         # This allows automatic reduction of the baseline file when needed.
         entry: sh -ec "pydoclint . && pydoclint --generate-baseline=1 ."
         pass_filenames: false
 
   - repo: https://github.com/pycqa/pylint.git
-    rev: v3.2.7
+    rev: v3.3.1
     hooks:
       - id: pylint
         args:

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,