Merge pull request #45 from taskbadger/sk/refresh-tooling

Refresh lint/format tooling

Author: snopoke

.pre-commit-config.yaml

@@ -3,13 +3,13 @@ default_language_version:
   python: python3.12
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.2.1
+    rev: v0.15.12
     hooks:
-      - id: ruff
+      - id: ruff-check
         args: [ --fix ]
       - id: ruff-format
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.4.0
+    rev: v6.0.0
     hooks:
       - id: check-added-large-files
         args: [ --maxkb=2048 ]

pyproject.toml

@@ -57,8 +57,6 @@ documentation = "https://docs.taskbadger.net/"
 
 [dependency-groups]
 dev = [
-    "black",
-    "isort",
     "pre-commit",
     "pytest",
     "pytest-httpx",
@@ -100,4 +98,5 @@ skip-magic-trailing-comma = false
 line-ending = "auto"
 
 [tool.ruff.lint.per-file-ignores]
-"taskbadger/internal/*" = ["E501"]
+# taskbadger/internal/ is generated by openapi-python-client; lint is best-effort
+"taskbadger/internal/*" = ["E501", "UP", "F"]

taskbadger/celery.py

@@ -263,7 +263,7 @@ def _maybe_create_task(signal_sender):
             inner_task = celery.current_app.tasks.get(task_name)
         items = signal_sender.request.kwargs.get("it", [])
         # Convert to list if needed for counting and potential recording
-        items_list = list(items) if not isinstance(items, (list, tuple)) else items
+        items_list = list(items) if not isinstance(items, list | tuple) else items
         item_count = len(items_list)
         # Append canvas type and item count to task name
         task_name = f"{task_name} ({canvas_type} {item_count})"

taskbadger/cli/utils.py

@@ -1,6 +1,5 @@
 import json
 from enum import Enum
-from typing import Optional
 
 import typer
 from rich import print
@@ -29,7 +28,7 @@ def get_actions(action_def: tuple[str, str, str]) -> list[Action]:
     return []
 
 
-def merge_kv_json(metadata_kv: Optional[list[str]], metadata_json: str) -> dict:
+def merge_kv_json(metadata_kv: list[str] | None, metadata_json: str) -> dict:
     metadata = {}
     for kv in metadata_kv or []:
         k, v = kv.strip().split("=", 1)

taskbadger/cli_main.py

@@ -72,15 +72,15 @@ def info(ctx: typer.Context):
     @app.callback()
     def main(
         ctx: typer.Context,
-        org: Optional[str] = typer.Option(
+        org: str | None = typer.Option(
             None,
             "--org",
             "-o",
             metavar="TASKBADGER_ORG",
             show_default=False,
             help="Organization Slug. This will override values from the config file and environment variables.",
         ),
-        project: Optional[str] = typer.Option(
+        project: str | None = typer.Option(
             None,
             "--project",
             "-p",

taskbadger/mug.py

@@ -1,16 +1,16 @@
 import dataclasses
+from collections.abc import Callable
 from contextlib import ContextDecorator
 from contextvars import ContextVar
 from copy import deepcopy
-from typing import Callable, Optional, Union
 
 from taskbadger.internal import AuthenticatedClient
 from taskbadger.systems import System
 
 _local = ContextVar("taskbadger_client")
 
 
-Callback = Union[str, Callable[[dict], Optional[dict]]]
+Callback = str | Callable[[dict], dict | None]
 
 
 @dataclasses.dataclass
@@ -47,7 +47,7 @@ class Session(ContextDecorator):
     def __init__(self):
         self._session = None
 
-    def __enter__(self) -> Union[AuthenticatedClient, None]:
+    def __enter__(self) -> AuthenticatedClient | None:
         if Badger.is_configured():
             self._session = Badger.current.session()
             return self._session.__enter__()
@@ -144,7 +144,7 @@ def client(self) -> AuthenticatedClient:
     def scope(self) -> Scope:
         return self._scope
 
-    def call_before_create(self, task: dict) -> Optional[dict]:
+    def call_before_create(self, task: dict) -> dict | None:
         if self.settings and self.settings.before_create:
             return self.settings.before_create(task)
         return task

taskbadger/utils.py

@@ -5,7 +5,7 @@ def import_string(dotted_path):
     try:
         module_path, class_name = dotted_path.rsplit(".", 1)
     except ValueError as err:
-        raise ImportError("%s doesn't look like a module path" % dotted_path) from err
+        raise ImportError(f"{dotted_path} doesn't look like a module path") from err
 
     module = import_module(module_path)
 

tests/conftest.py

@@ -3,7 +3,7 @@
 from taskbadger.mug import Badger, Settings
 
 
-@pytest.fixture()
+@pytest.fixture
 def _bind_settings():
     Badger.current.bind(Settings("https://taskbadger.net", "token", "org", "proj"))
     yield

tests/test_celery_system_integration.py

@@ -39,7 +39,7 @@ def _wait_for_mock_calls(mock_obj, expected_count, timeout=5):
         time.sleep(0.05)
 
 
-@pytest.fixture()
+@pytest.fixture
 def _bind_settings_with_system():
     systems = [CelerySystemIntegration()]
     Badger.current.bind(

tests/test_cli_config.py

@@ -21,7 +21,7 @@ def mock_config_location():
         os.remove(config_path)
 
 
-@pytest.fixture()
+@pytest.fixture
 def _mock_config(mock_config_location):
     config = Config(
         organization_slug="test_org",