Single startswith() call instead of multiple ones (zarr-developers#1556)

It's faster and probably more readable. Co-authored-by: Davis Bennett <[email protected]> Co-authored-by: Joe Hamman <[email protected]>
DimitriPapadopoulos · Dec 7, 2023 · 40a6e81 · 40a6e81
1 parent 10dee6b
commit 40a6e81
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/zarr/_storage/store.py b/zarr/_storage/store.py
@@ -221,9 +221,8 @@ def _validate_key(self, key: str):
             )
 
         if (
-            not key.startswith("data/")
-            and (not key.startswith("meta/"))
-            and (not key == "zarr.json")
+            not key.startswith(("data/", "meta/"))
+            and key != "zarr.json"
             # TODO: Possibly allow key == ".zmetadata" too if we write a
             #       consolidated metadata spec corresponding to this?
         ):