refresh lint/format tooling

- Bump pre-commit hooks: ruff-pre-commit v0.2.1 -> v0.15.12, pre-commit-hooks v4.4.0 -> v6.0.0 (both 2+ years stale)
- Update ruff hook id from `ruff` (legacy alias) to `ruff-check` per the newer ruff-pre-commit naming
- Apply lint findings the bumped ruff surfaces: PT001 (drop () from @pytest.fixture decorators), UP031 (percent-format -> f-string in utils.py), PT030 (add `match` to pytest.warns(DeprecationWarning))
- Drop black and isort from the dev dependency group; ruff and ruff-format already cover both

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

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

@@ -58,8 +58,6 @@ documentation = "https://docs.taskbadger.net/"
 
 [dependency-groups]
 dev = [
-    "black",
-    "isort",
     "pre-commit",
     "pytest",
     "pytest-httpx",

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",

tests/test_project_key.py

@@ -102,7 +102,7 @@ def test_init_legacy_key_emits_deprecation_warning(self):
             init("org", "project", "legacy-token")
 
     def test_init_legacy_key_still_works(self):
-        with pytest.warns(DeprecationWarning):
+        with pytest.warns(DeprecationWarning, match="Legacy API keys are deprecated"):
             init("org", "project", "legacy-token")
         settings = Badger.current.settings
         assert settings.organization_slug == "org"
@@ -152,7 +152,7 @@ def test_project_key_is_valid(self):
 runner = CliRunner()
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_config_location():
     config_path = Path(__file__).parent / "_mock_config_project_key"
     with mock.patch("taskbadger.config._get_config_path", return_value=config_path):

tests/test_sdk.py

@@ -24,24 +24,24 @@ def _init_skd():
         init("org", "project", "token")
 
 
-@pytest.fixture()
+@pytest.fixture
 def settings():
     return Badger.current.settings
 
 
-@pytest.fixture()
+@pytest.fixture
 def patched_get():
     with mock.patch("taskbadger.sdk.task_get.sync") as get:
         yield get
 
 
-@pytest.fixture()
+@pytest.fixture
 def patched_create():
     with mock.patch("taskbadger.sdk.task_create.sync_detailed") as create:
         yield create
 
 
-@pytest.fixture()
+@pytest.fixture
 def patched_update():
     with mock.patch("taskbadger.sdk.task_partial_update.sync_detailed") as update:
         yield update