From 4cfbbaea6aa0c153bc794fb4f342b9007350da77 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Thu, 31 Oct 2024 14:11:00 +0000 Subject: [PATCH] chore: pre-commit autoupdate (#256) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * chore: pre-commit autoupdate updates: - [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](https://github.com/pre-commit/pre-commit-hooks/compare/v4.6.0...v5.0.0) - [github.com/tox-dev/tox-ini-fmt: 1.3.1 → 1.4.1](https://github.com/tox-dev/tox-ini-fmt/compare/1.3.1...1.4.1) - [github.com/astral-sh/ruff-pre-commit: v0.6.3 → v0.6.9](https://github.com/astral-sh/ruff-pre-commit/compare/v0.6.3...v0.6.9) - [github.com/jsh9/pydoclint: 0.5.6 → 0.5.9](https://github.com/jsh9/pydoclint/compare/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 --- .pre-commit-config.yaml | 10 +++++----- src/ansible_dev_environment/output.py | 2 +- src/ansible_dev_environment/subcommands/installer.py | 4 ++-- src/ansible_dev_environment/tree.py | 2 +- src/ansible_dev_environment/utils.py | 2 +- tests/unit/test_installer.py | 2 +- 6 files changed, 11 insertions(+), 11 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 3013a84..d145527 100644 --- a/.pre-commit-config.yaml +++ b/.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,7 +72,7 @@ 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. @@ -80,7 +80,7 @@ repos: pass_filenames: false - repo: https://github.com/pycqa/pylint.git - rev: v3.2.7 + rev: v3.3.1 hooks: - id: pylint args: diff --git a/src/ansible_dev_environment/output.py b/src/ansible_dev_environment/output.py index 8f64638..0b17854 100644 --- a/src/ansible_dev_environment/output.py +++ b/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, diff --git a/src/ansible_dev_environment/subcommands/installer.py b/src/ansible_dev_environment/subcommands/installer.py index 323b4c5..42d01e1 100644 --- a/src/ansible_dev_environment/subcommands/installer.py +++ b/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) diff --git a/src/ansible_dev_environment/tree.py b/src/ansible_dev_environment/tree.py index dcaeefc..29ad0f3 100644 --- a/src/ansible_dev_environment/tree.py +++ b/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 diff --git a/src/ansible_dev_environment/utils.py b/src/ansible_dev_environment/utils.py index e8154c7..e4da9b1 100644 --- a/src/ansible_dev_environment/utils.py +++ b/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, diff --git a/tests/unit/test_installer.py b/tests/unit/test_installer.py index bd49f58..56339fe 100644 --- a/tests/unit/test_installer.py +++ b/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,