diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index ed53ec5a..c926c607 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -5,16 +5,16 @@ ci: repos: - repo: https://github.com/psf/black - rev: 24.8.0 + rev: 24.10.0 hooks: - id: black - repo: https://github.com/astral-sh/ruff-pre-commit - rev: v0.5.7 + rev: v0.7.3 hooks: - id: ruff args: [--fix, --exit-non-zero-on-fix] - repo: https://github.com/pre-commit/pre-commit-hooks - rev: v4.6.0 + rev: v5.0.0 hooks: - id: check-yaml - id: debug-statements diff --git a/nailgun/entity_mixins.py b/nailgun/entity_mixins.py index 4e50e9c4..91027fff 100644 --- a/nailgun/entity_mixins.py +++ b/nailgun/entity_mixins.py @@ -134,7 +134,7 @@ def raise_task_timeout(): # pragma: no cover except KeyboardInterrupt: # raise_task_timeout will raise a KeyboardInterrupt when the timeout # expires. Catch the exception and raise TaskTimedOutError - raise TaskTimedOutError( # noqa: TRY200 - Not raising from KeyBoardInterrupt + raise TaskTimedOutError( # noqa: B904 - Not raising from KeyBoardInterrupt f"Timed out polling task {task_id}. Task information: {task_info}", task_id ) finally: diff --git a/pyproject.toml b/pyproject.toml index d668c4e3..222f5b1a 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -31,6 +31,7 @@ select = [ "B014", # Exception handler with duplicate exception "B023", # Function definition does not bind loop variable {name} "B026", # Star-arg unpacking after a keyword argument is strongly discouraged + "B904", # Use raise from to specify exception cause "BLE001", # Using bare except clauses is prohibited "C", # complexity "C4", # flake8-comprehensions @@ -74,7 +75,6 @@ select = [ "T100", # Trace found: {name} used "T20", # flake8-print "TRY004", # Prefer TypeError exception for invalid type - "TRY200", # Use raise from to specify exception cause "TRY302", # Remove exception handler; error is immediately re-raised "PLR0911", # Too many return statements ({returns} > {max_returns}) "PLR0912", # Too many branches ({branches} > {max_branches})