Update pre-commit hooks + mypy fixes

Author: dstansby

.pre-commit-config.yaml

@@ -6,7 +6,7 @@ ci:
 default_stages: [pre-commit, pre-push]
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.11.9
+    rev: v0.11.13
     hooks:
       - id: ruff
         args: ["--fix", "--show-fixes"]
@@ -22,7 +22,7 @@ repos:
       - id: check-yaml
       - id: trailing-whitespace
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.15.0
+    rev: v1.16.0
     hooks:
       - id: mypy
         files: src|tests

src/zarr/core/buffer/core.py

@@ -93,7 +93,7 @@ def ravel(self, order: Literal["K", "A", "C", "F"] = ...) -> Self: ...
 
     def all(self) -> bool: ...
 
-    def __eq__(self, other: object) -> Self:  # type: ignore[explicit-override, override]
+    def __eq__(self, other: object) -> Self:  # type: ignore[override]
         """Element-wise equal
 
         Notes

src/zarr/core/dtype/npy/bytes.py

@@ -176,14 +176,14 @@ def _check_native_dtype(
         Bool
             True if the dtype matches, False otherwise.
         """
-        return cls.dtype_cls is type(dtype) and dtype.fields is None  # type: ignore[has-type]
+        return cls.dtype_cls is type(dtype) and dtype.fields is None
 
     @classmethod
     def from_native_dtype(cls, dtype: TBaseDType) -> Self:
         if cls._check_native_dtype(dtype):
             return cls(length=dtype.itemsize)
         raise DataTypeValidationError(
-            f"Invalid data type: {dtype}. Expected an instance of {cls.dtype_cls}"  # type: ignore[has-type]
+            f"Invalid data type: {dtype}. Expected an instance of {cls.dtype_cls}"
         )
 
     def to_native_dtype(self) -> np.dtypes.VoidDType[int]:

src/zarr/core/dtype/npy/structured.py

@@ -52,7 +52,7 @@ def _check_native_dtype(cls, dtype: TBaseDType) -> TypeGuard[np.dtypes.VoidDType
         TypeGuard[np.dtypes.VoidDType]
             True if the dtype matches, False otherwise.
         """
-        return isinstance(dtype, cls.dtype_cls) and dtype.fields is not None  # type: ignore[has-type]
+        return isinstance(dtype, cls.dtype_cls) and dtype.fields is not None
 
     @classmethod
     def from_native_dtype(cls, dtype: TBaseDType) -> Self:
@@ -68,7 +68,7 @@ def from_native_dtype(cls, dtype: TBaseDType) -> Self:
 
             return cls(fields=tuple(fields))
         raise DataTypeValidationError(
-            f"Invalid data type: {dtype}. Expected an instance of {cls.dtype_cls}"  # type: ignore[has-type]
+            f"Invalid data type: {dtype}. Expected an instance of {cls.dtype_cls}"
         )
 
     def to_native_dtype(self) -> np.dtypes.VoidDType[int]:

src/zarr/core/dtype/npy/time.py

@@ -230,7 +230,7 @@ def _from_json_v2(cls, data: DTypeJSON) -> Self:
             return cls.from_native_dtype(np.dtype(name))
         msg = (
             f"Invalid JSON representation of {cls.__name__}. Got {data!r}, expected a string "
-            f"representation of an instance of {cls.dtype_cls}"  # type: ignore[has-type]
+            f"representation of an instance of {cls.dtype_cls}"
         )
         raise DataTypeValidationError(msg)
 
@@ -318,7 +318,7 @@ def _from_json_v2(cls, data: DTypeJSON) -> Self:
             return cls.from_native_dtype(np.dtype(name))
         msg = (
             f"Invalid JSON representation of {cls.__name__}. Got {data!r}, expected a string "
-            f"representation of an instance of {cls.dtype_cls}"  # type: ignore[has-type]
+            f"representation of an instance of {cls.dtype_cls}"
         )
         raise DataTypeValidationError(msg)
 

src/zarr/storage/_memory.py

@@ -143,7 +143,9 @@ async def delete(self, key: str) -> None:
         except KeyError:
             logger.debug("Key %s does not exist.", key)
 
-    async def set_partial_values(self, key_start_values: Iterable[tuple[str, int, bytes]]) -> None:
+    async def set_partial_values(
+        self, key_start_values: Iterable[tuple[str, int, bytes | bytearray | memoryview[int]]]
+    ) -> None:
         # docstring inherited
         raise NotImplementedError
 

src/zarr/storage/_zip.py

@@ -222,7 +222,9 @@ async def set(self, key: str, value: Buffer) -> None:
         with self._lock:
             self._set(key, value)
 
-    async def set_partial_values(self, key_start_values: Iterable[tuple[str, int, bytes]]) -> None:
+    async def set_partial_values(
+        self, key_start_values: Iterable[tuple[str, int, bytes | bytearray | memoryview[int]]]
+    ) -> None:
         raise NotImplementedError
 
     async def set_if_not_exists(self, key: str, value: Buffer) -> None:

tests/test_config.py

@@ -26,6 +26,7 @@
 from zarr.core.codec_pipeline import BatchedCodecPipeline
 from zarr.core.config import BadConfigError, config
 from zarr.core.dtype import Int8, VariableLengthUTF8
+from zarr.core.dtype.wrapper import TBaseDType, TBaseScalar
 from zarr.core.indexing import SelectorTuple
 from zarr.registry import (
     fully_qualified_name,
@@ -329,9 +330,9 @@ async def test_default_codecs(dtype_category: str) -> None:
     """
     Test that the default compressors are sensitive to the current setting of the config.
     """
-    zdtype: ZDType[Any, Any]
+    zdtype: ZDType[TBaseDType, TBaseScalar]
     if dtype_category == "variable-length-string":
-        zdtype = VariableLengthUTF8()
+        zdtype = VariableLengthUTF8()  # type: ignore[assignment]
     else:
         zdtype = Int8()
     expected_compressors = (GzipCodec(),)