Update hooks and address issues; add 3.14 to test matrix (#321)

Author: stefmolin

.github/workflows/ci.yml

@@ -44,7 +44,7 @@ jobs:
         fail-fast: false
         matrix:
             os: [macos-latest, ubuntu-latest, windows-latest]
-            python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+            python-version: ["3.9", "3.10", "3.11", "3.12", "3.13", "3.14"]
 
     env:
       MPLBACKEND: Agg  # non-interactive backend for matplotlib

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v5.0.0
+    rev: v6.0.0
     hooks:
       - id: check-added-large-files
       - id: check-merge-conflict
@@ -29,7 +29,7 @@ repos:
         exclude: (\.(svg|png|pdf)$)|(CODE_OF_CONDUCT.md)
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.2
+    rev: v0.14.2
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix, --show-fixes]
@@ -41,14 +41,14 @@ repos:
       - id: numpydoc-validation
         exclude: (tests|docs)/.*
 
-  - repo: https://github.com/econchick/interrogate
-    rev: 1.7.0
+  - repo: https://github.com/stefmolin/docstringify
+    rev: 1.1.1
     hooks:
-      - id: interrogate
+      - id: docstringify
         files: tests/.*
 
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: v2.6.0
+    rev: v2.11.0
     hooks:
       - id: pyproject-fmt
         args: [--keep-full-version, --no-print-diff]

pyproject.toml

@@ -36,6 +36,7 @@ classifiers = [
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3.13",
+  "Programming Language :: Python :: 3.14",
   "Topic :: Scientific/Engineering :: Artificial Intelligence",
   "Topic :: Scientific/Engineering :: Visualization",
 ]
@@ -173,31 +174,6 @@ markers = [
   "shapes: Run tests related to shapes.",
 ]
 
-[tool.interrogate]
-ignore-init-method = true
-ignore-init-module = false
-ignore-magic = false
-ignore-semiprivate = false
-ignore-private = false
-ignore-property-decorators = false
-ignore-module = false
-ignore-nested-functions = false
-ignore-nested-classes = true
-ignore-setters = false
-fail-under = 100
-exclude = [
-  "setup.py",
-  "docs",
-  "build",
-]
-verbose = 2
-quiet = false
-color = true
-omit-covered-files = false
-# TODO: revisit this later when I add badges to the README
-# generate-badge = "."
-# badge-format = "svg"
-
 [tool.numpydoc_validation]
 checks = [
   "all",  # report on all checks

tests/data/test_dataset.py

@@ -54,7 +54,7 @@ def test_validate_data_missing_columns(self, starter_shapes_dir):
 
         data = pd.read_csv(starter_shapes_dir / 'dino.csv').rename(columns={'x': 'a'})
 
-        with pytest.raises(ValueError, match='Columns "x" and "y" are required.'):
+        with pytest.raises(ValueError, match='Columns "x" and "y" are required'):
             _ = Dataset('dino', data)
 
     def test_validate_data_fix_column_casing(self, starter_shapes_dir):

tests/plotting/test_animation.py

@@ -120,6 +120,6 @@ def test_invalid_easing_step(ease_function, invalid_step):
     ease_func = getattr(animation, ease_function)
 
     with pytest.raises(
-        ValueError, match='Step must be an integer or float, between 0 and 1.'
+        ValueError, match='Step must be an integer or float, between 0 and 1'
     ):
         ease_func(invalid_step)

tests/shapes/bases/test_shape.py

@@ -15,10 +15,14 @@ def test_is_abc(self):
             _ = Shape()
 
         class NewShape(Shape):
+            """A test shape."""
+
             def distance(self, x, y):
+                """Calculate the distance from the shape to the point."""
                 return super().distance(x, y)
 
             def plot(self, ax=None):
+                """Plot the shape."""
                 return super().plot(ax)
 
         with pytest.raises(NotImplementedError):
@@ -31,10 +35,14 @@ def test_repr(self):
         """Test that the __repr__() method is working."""
 
         class NewShape(Shape):
+            """A test shape."""
+
             def distance(self, x, y):  # pragma: no cover
+                """Calculate the distance from the shape to the point."""
                 return x, y
 
             def plot(self, ax):  # pragma: no cover
+                """Plot the shape."""
                 return ax
 
         new_shape = NewShape()

tests/test_cli.py

@@ -33,7 +33,7 @@ def test_cli_version(capsys):
 
 def test_cli_bad_shape():
     """Test that invalid target shapes raise a ValueError."""
-    with pytest.raises(ValueError, match='No valid target shapes were provided.'):
+    with pytest.raises(ValueError, match='No valid target shapes were provided'):
         cli.main(['--start-shape=dino', '--target-shape=does-not-exist'])