[pre-commit.ci] pre-commit autoupdate

.pre-commit-config.yaml

@@ -34,7 +34,7 @@ repos:
           - prettier-plugin-toml
 
   - repo: https://github.com/psf/black.git
-    rev: 24.8.0
+    rev: 24.10.0
     hooks:
       - id: black
         language_version: python3
@@ -45,14 +45,14 @@ repos:
       - id: toml-sort-fix
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.6.5"
+    rev: "v0.7.0"
     hooks:
       - id: ruff
         args:
           - "--exit-non-zero-on-fix"
 
   - repo: https://github.com/streetsidesoftware/cspell-cli
-    rev: v8.13.3
+    rev: v8.15.2
     hooks:
       - id: cspell
         name: Spell check with cspell
@@ -67,7 +67,7 @@ repos:
           )$
 
   - repo: https://github.com/pre-commit/pre-commit-hooks.git
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       # Side-effects:
       - id: trailing-whitespace
@@ -98,7 +98,7 @@ repos:
       - id: check-json5
 
   - repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.41.0
+    rev: v0.42.0
     hooks:
       - id: markdownlint
         exclude: >
@@ -147,18 +147,18 @@ repos:
           - flake8-docstrings # uses pydocstyle
 
   - repo: https://github.com/jsh9/pydoclint
-    rev: 0.5.7
+    rev: 0.5.9
     hooks:
       - id: pydoclint
 
   - repo: https://github.com/asottile/pyupgrade
     # keep it after flake8
-    rev: v3.17.0
+    rev: v3.18.0
     hooks:
       - id: pyupgrade
         args: ["--py310-plus"]
   - repo: https://github.com/pre-commit/mirrors-mypy.git
-    rev: v1.11.2
+    rev: v1.12.1
     hooks:
       - id: mypy
         additional_dependencies:
@@ -175,7 +175,7 @@ repos:
           - types-typed-ast
 
   - repo: https://github.com/pycqa/pylint.git
-    rev: v3.2.7
+    rev: v3.3.1
     hooks:
       - id: pylint
         args:

src/ansible_navigator/actions/_actions.py

@@ -197,7 +197,7 @@ def run_stdout(package: str, action: str, *args: Any, **_kwargs: dict[str, Any])
     :param args: The arguments passed to the action's run_stdout method
     :param _kwargs: The keyword arguments passed to the action's run_stdout method
     :returns: The outcome of running the action's ``run_stdout()`` method
-    """  # noqa: D402 # Refers to the action's run_stdout in the first line, not this function
+    """  # Refers to the action's run_stdout in the first line, not this function
     action_cls = get(package, action)
     args = args[0]
     return action_cls(args).run_stdout()

tests/conftest.py

@@ -112,7 +112,7 @@ def fixture_small_image_name() -> str:
     return small_image_name()
 
 
-@pytest.fixture()
+@pytest.fixture
 def locked_directory(tmpdir: str) -> Generator[str]:
     """Directory without read-write for throwing errors.
 
@@ -136,7 +136,7 @@ def pullable_image(valid_container_engine: str) -> Generator[str]:
     subprocess.run([valid_container_engine, "image", "rm", image], check=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def _patch_curses(monkeypatch: pytest.MonkeyPatch) -> None:
     """Patch curses so it doesn't traceback during tests.
 
@@ -158,7 +158,7 @@ def _settings_samples() -> tuple[str, str]:
     return commented, uncommented
 
 
-@pytest.fixture()
+@pytest.fixture
 def settings_env_var_to_full(
     settings_samples: tuple[str, str],
     tmp_path: Path,
@@ -196,7 +196,7 @@ def settings_env_var_to_full(
     return settings_path, settings_contents
 
 
-@pytest.fixture()
+@pytest.fixture
 def test_dir_fixture_dir(request: pytest.FixtureRequest) -> Path:
     """Provide the fixture directory for a given test directory.
 
@@ -302,7 +302,7 @@ def _cmd_in_tty(
     return result[m_stdout].decode("utf-8"), result[m_stderr].decode("utf-8"), proc.returncode
 
 
-@pytest.fixture()
+@pytest.fixture
 def cmd_in_tty() -> Generator[Callable[..., tuple[str, str, int]]]:
     """Provide the cmd in tty function as a fixture.
 
@@ -415,7 +415,7 @@ def pytest_unconfigure(config: pytest.Config) -> None:
         os.environ[key] = value
 
 
-@pytest.fixture()
+@pytest.fixture
 def skip_if_already_failed(
     request: pytest.FixtureRequest,
     failed: set[str] = set(),

tests/integration/conftest.py

@@ -27,7 +27,7 @@
 EXECUTION_MODES = ["interactive", "stdout"]
 
 
-@pytest.fixture()
+@pytest.fixture
 def action_run_stdout() -> Generator[type[ActionRunTest]]:
     """Create a fixture for ActionRunTest.
 
@@ -135,7 +135,7 @@ def run(self) -> tuple[str, str, int]:
         return cmd_in_tty(self.to_cmdline(), cwd=self.cwd)
 
 
-@pytest.fixture()
+@pytest.fixture
 def cli_runner(request: pytest.FixtureRequest) -> CliRunner:
     """Create a fixture for the cli runner.
 

tests/unit/configuration_subsystem/conftest.py

@@ -113,7 +113,7 @@ def fixture_generate_config() -> Callable[..., GenerateConfigResponse]:
     return _generate_config
 
 
-@pytest.fixture()
+@pytest.fixture
 def _ansible_version(monkeypatch: pytest.MonkeyPatch) -> None:
     """Path the ansible --version call to avoid the subprocess calls.
 
@@ -146,7 +146,7 @@ def _schema_dict() -> SettingsSchemaType:
     return SettingsSchemaType(schema)
 
 
-@pytest.fixture()
+@pytest.fixture
 def schema_dict_all_required(schema_dict: SettingsSchemaType) -> SettingsSchemaType:
     """Provide the json schema as a dictionary with all properties required.
 

tests/unit/utils/conftest.py

@@ -8,7 +8,7 @@
 from ansible_navigator.utils.key_value_store import KeyValueStore
 
 
-@pytest.fixture()
+@pytest.fixture
 def empty_kvs(tmp_path: Path) -> Generator[KeyValueStore, None, None]:
     """Give us a temporary, empty KeyValueStore.
 
@@ -19,7 +19,7 @@ def empty_kvs(tmp_path: Path) -> Generator[KeyValueStore, None, None]:
     yield KeyValueStore(database_path)
 
 
-@pytest.fixture()
+@pytest.fixture
 def kvs(tmp_path: Path) -> Generator[KeyValueStore, None, None]:
     """Give us a temporary KeyValueStore with some data.