Unnecessary comprehension (#1996)

* Unnecessary comprehension

`all` can take a generator.

* Unnecessary comprehension

`tuple` can take a generator.

Author: DimitriPapadopoulos

src/zarr/indexing.py

@@ -1023,7 +1023,7 @@ def __init__(self, selection: CoordinateSelection, shape: ChunkCoords, chunk_gri
         # flatten selection
         selection_broadcast = tuple(dim_sel.reshape(-1) for dim_sel in selection_broadcast)
         chunks_multi_index_broadcast = tuple(
-            [dim_chunks.reshape(-1) for dim_chunks in chunks_multi_index_broadcast]
+            dim_chunks.reshape(-1) for dim_chunks in chunks_multi_index_broadcast
         )
 
         # ravel chunk indices

src/zarr/metadata.py

@@ -436,7 +436,7 @@ def update_attributes(self, attributes: dict[str, JSON]) -> Self:
 def parse_dimension_names(data: None | Iterable[str]) -> tuple[str, ...] | None:
     if data is None:
         return data
-    elif all([isinstance(x, str) for x in data]):
+    elif all(isinstance(x, str) for x in data):
         return tuple(data)
     else:
         msg = f"Expected either None or a iterable of str, got {type(data)}"