Upgrade ruff to 0.9.1 (#2703)

DimitriPapadopoulos · web-flow · commit 280daac770cf · 2025-01-16T16:25:02.000Z
* Apply ruff rule RUF100

RUF100 Unused `noqa` directive (unused: `E402`)

Otherwise, latest ruff 0.9.1 complains.

* Enable ruff/flake8-implicit-str-concat rules ISC001 and ISC002

Starting with ruff 0.9.1, they are documented to be compatible with the
ruff formatter, if they are both enabled.

* Run `ruff format`

* Upgrade ruff to 0.9.1
diff --git a/.pre-commit-config.yaml b/.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.8.6
+    rev: v0.9.1
     hooks:
     - id: ruff
       args: ["--fix", "--show-fixes"]
diff --git a/pyproject.toml b/pyproject.toml
@@ -326,8 +326,6 @@ ignore = [
     "Q003",
     "COM812",
     "COM819",
-    "ISC001",
-    "ISC002",
 ]
 
 [tool.ruff.lint.extend-per-file-ignores]
diff --git a/src/zarr/codecs/sharding.py b/src/zarr/codecs/sharding.py
@@ -86,9 +86,9 @@ async def get(
         self, prototype: BufferPrototype, byte_range: ByteRequest | None = None
     ) -> Buffer | None:
         assert byte_range is None, "byte_range is not supported within shards"
-        assert (
-            prototype == default_buffer_prototype()
-        ), f"prototype is not supported within shards currently. diff: {prototype} != {default_buffer_prototype()}"
+        assert prototype == default_buffer_prototype(), (
+            f"prototype is not supported within shards currently. diff: {prototype} != {default_buffer_prototype()}"
+        )
         return self.shard_dict.get(self.chunk_coords)
 
 
diff --git a/src/zarr/core/indexing.py b/src/zarr/core/indexing.py
@@ -289,9 +289,9 @@ def is_pure_orthogonal_indexing(selection: Selection, ndim: int) -> TypeGuard[Or
 def get_chunk_shape(chunk_grid: ChunkGrid) -> ChunkCoords:
     from zarr.core.chunk_grids import RegularChunkGrid
 
-    assert isinstance(
-        chunk_grid, RegularChunkGrid
-    ), "Only regular chunk grid is supported, currently."
+    assert isinstance(chunk_grid, RegularChunkGrid), (
+        "Only regular chunk grid is supported, currently."
+    )
     return chunk_grid.chunk_shape
 
 
diff --git a/src/zarr/core/metadata/v3.py b/src/zarr/core/metadata/v3.py
@@ -373,9 +373,9 @@ def inner_codecs(self) -> tuple[Codec, ...]:
     def get_chunk_spec(
         self, _chunk_coords: ChunkCoords, array_config: ArrayConfig, prototype: BufferPrototype
     ) -> ArraySpec:
-        assert isinstance(
-            self.chunk_grid, RegularChunkGrid
-        ), "Currently, only regular chunk grid is supported"
+        assert isinstance(self.chunk_grid, RegularChunkGrid), (
+            "Currently, only regular chunk grid is supported"
+        )
         return ArraySpec(
             shape=self.chunk_grid.chunk_shape,
             dtype=self.dtype,
diff --git a/tests/test_properties.py b/tests/test_properties.py
@@ -4,11 +4,11 @@
 
 pytest.importorskip("hypothesis")
 
-import hypothesis.extra.numpy as npst  # noqa: E402
-import hypothesis.strategies as st  # noqa: E402
-from hypothesis import given  # noqa: E402
+import hypothesis.extra.numpy as npst
+import hypothesis.strategies as st
+from hypothesis import given
 
-from zarr.testing.strategies import arrays, basic_indices, numpy_arrays, zarr_formats  # noqa: E402
+from zarr.testing.strategies import arrays, basic_indices, numpy_arrays, zarr_formats
 
 
 @given(data=st.data(), zarr_format=zarr_formats)

Original file line number	Diff line number	Diff line change
`@@ -326,8 +326,6 @@ ignore = [`
`326`	`326`	`"Q003",`
`327`	`327`	`"COM812",`
`328`	`328`	`"COM819",`
`329`		`- "ISC001",`
`330`		`- "ISC002",`
`331`	`329`	`]`
`332`	`330`
`333`	`331`	`[tool.ruff.lint.extend-per-file-ignores]`