Use removeprefix rather than replace to avoid separator deletion

zarr-developers · Jan 30, 2025 · 90812f5 · 90812f5
1 parent fc08f31
commit 90812f5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/changes/2778.bugfix.rst b/changes/2778.bugfix.rst
@@ -0,0 +1 @@
+Use removeprefix rather than replace when removing filename prefixes in `FsspecStore.list`
diff --git a/src/zarr/storage/_fsspec.py b/src/zarr/storage/_fsspec.py
@@ -341,7 +341,7 @@ async def set_partial_values(
     async def list(self) -> AsyncIterator[str]:
         # docstring inherited
         allfiles = await self.fs._find(self.path, detail=False, withdirs=False)
-        for onefile in (a.replace(self.path + "/", "") for a in allfiles):
+        for onefile in (a.removeprefix(self.path + "/") for a in allfiles):
             yield onefile
 
     async def list_dir(self, prefix: str) -> AsyncIterator[str]:
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Use removeprefix rather than replace when removing filename prefixes in `FsspecStore.list`