Skip to content

Commit

Permalink
_bulk_delete must always return a list, even when pathlist is empty (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
ghislainp authored Mar 26, 2024
1 parent 11ec756 commit 7ccc0a5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion s3fs/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -1954,7 +1954,7 @@ async def _bulk_delete(self, pathlist, **kwargs):
Must have 0 < len <= 1000
"""
if not pathlist:
return
return []
buckets = {self.split_path(path)[0] for path in pathlist}
if len(buckets) > 1:
raise ValueError("Bulk delete files should refer to only one bucket")
Expand Down

0 comments on commit 7ccc0a5

Please sign in to comment.