Skip to content

Commit

Permalink
Fix key to manifest value on windows?
Browse files Browse the repository at this point in the history
  • Loading branch information
LDeakin committed Sep 22, 2024
1 parent 2f64699 commit 9dc326e
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,10 @@ struct ChunkManifestJsonImplStorageTransformer<TStorage: ?Sized> {

impl<TStorage: ?Sized> ChunkManifestJsonImplStorageTransformer<TStorage> {
fn key_to_manifest_value(&self, key: &StoreKey) -> Option<&ChunkManifestValue> {
let root = strip_root_prefix(self.path.as_str());
let mut root = strip_root_prefix(self.path.as_str());
if cfg!(windows) {
root = root.replace('\\', "/");

Check warning on line 202 in zarrs/src/array/storage_transformer/chunk_manifest_json/chunk_manifest_storage_transformer.rs

View check run for this annotation

Codecov / codecov/patch

zarrs/src/array/storage_transformer/chunk_manifest_json/chunk_manifest_storage_transformer.rs#L202

Added line #L202 was not covered by tests
}
let relative_key = key
.as_str()
.strip_prefix(&root)
Expand Down

0 comments on commit 9dc326e

Please sign in to comment.