Enable some ruff rules (RUF) and fix issues (#1869)

* Enable some ruff rules (RUF)

* Apply ruff rule RUF010

RUF010 Use explicit conversion flag

* Apply ruff rule RUF019

RUF019 Unnecessary key check before dictionary access

* Apply ruff rule RUF100

RUF100 Unused `noqa` directive

Author: DimitriPapadopoulos

pyproject.toml

@@ -163,6 +163,18 @@ extend-exclude = [
     "docs"
 ]
 
+[tool.ruff.lint]
+extend-select = [
+    "RUF"
+]
+ignore = [
+    "RUF003",
+    "RUF005",
+    "RUF009",
+    "RUF012",
+    "RUF015",
+]
+
 [tool.mypy]
 python_version = "3.10"
 ignore_missing_imports = true

src/zarr/__init__.py

@@ -3,11 +3,11 @@
 from typing import Union
 
 import zarr.codecs  # noqa: F401
-from zarr.array import Array, AsyncArray  # noqa: F401
+from zarr.array import Array, AsyncArray
 from zarr.array_v2 import ArrayV2
 from zarr.config import config  # noqa: F401
-from zarr.group import AsyncGroup, Group  # noqa: F401
-from zarr.store import (  # noqa: F401
+from zarr.group import AsyncGroup, Group
+from zarr.store import (
     StoreLike,
     make_store_path,
 )

src/zarr/codecs/pipeline.py

@@ -93,7 +93,7 @@ def from_list(cls, codecs: List[Codec]) -> CodecPipeline:
             array_array_codecs=tuple(
                 codec for codec in codecs if isinstance(codec, ArrayArrayCodec)
             ),
-            array_bytes_codec=[codec for codec in codecs if isinstance(codec, ArrayBytesCodec)][0],
+            array_bytes_codec=next(codec for codec in codecs if isinstance(codec, ArrayBytesCodec)),
             bytes_bytes_codecs=tuple(
                 codec for codec in codecs if isinstance(codec, BytesBytesCodec)
             ),

src/zarr/common.py

@@ -81,7 +81,7 @@ def parse_enum(data: JSON, cls: Type[E]) -> E:
         raise TypeError(f"Expected str, got {type(data)}")
     if data in enum_names(cls):
         return cls(data)
-    raise ValueError(f"Value must be one of {repr(list(enum_names(cls)))}. Got {data} instead.")
+    raise ValueError(f"Value must be one of {list(enum_names(cls))!r}. Got {data} instead.")
 
 
 @dataclass(frozen=True)

src/zarr/store/core.py

@@ -48,7 +48,7 @@ def __str__(self) -> str:
         return _dereference_path(str(self.store), self.path)
 
     def __repr__(self) -> str:
-        return f"StorePath({self.store.__class__.__name__}, {repr(str(self))})"
+        return f"StorePath({self.store.__class__.__name__}, {str(self)!r})"
 
     def __eq__(self, other: Any) -> bool:
         try:

src/zarr/store/local.py

@@ -83,7 +83,7 @@ def __str__(self) -> str:
         return f"file://{self.root}"
 
     def __repr__(self) -> str:
-        return f"LocalStore({repr(str(self))})"
+        return f"LocalStore({str(self)!r})"
 
     def __eq__(self, other: object) -> bool:
         return isinstance(other, type(self)) and self.root == other.root

src/zarr/store/memory.py

@@ -23,7 +23,7 @@ def __str__(self) -> str:
         return f"memory://{id(self._store_dict)}"
 
     def __repr__(self) -> str:
-        return f"MemoryStore({repr(str(self))})"
+        return f"MemoryStore({str(self)!r})"
 
     async def get(
         self, key: str, byte_range: Optional[Tuple[int, Optional[int]]] = None

src/zarr/store/remote.py

@@ -39,7 +39,7 @@ def __str__(self) -> str:
         return str(self.root)
 
     def __repr__(self) -> str:
-        return f"RemoteStore({repr(str(self))})"
+        return f"RemoteStore({str(self)!r})"
 
     def _make_fs(self) -> Tuple[AsyncFileSystem, str]:
         import fsspec

src/zarr/sync.py

@@ -84,7 +84,7 @@ def sync(
     if len(unfinished) > 0:
         raise asyncio.TimeoutError(f"Coroutine {coro} failed to finish in within {timeout}s")
     assert len(finished) == 1
-    return_result = list(finished)[0].result()
+    return_result = next(iter(finished)).result()
 
     if isinstance(return_result, BaseException):
         raise return_result

src/zarr/v2/_storage/absstore.py

@@ -58,7 +58,7 @@ class ABSStore(Store):
     -----
     In order to use this store, you must install the Microsoft Azure Storage SDK for Python,
     ``azure-storage-blob>=12.5.0``.
-    """  # noqa: E501
+    """
 
     def __init__(
         self,