Merge pull request #82 from LDeakin/v3_metadata_non_exhaustive

Mark `GroupMetadataV3` and `ArrayMetadataV3` as non-exhaustive
LDeakin · Oct 3, 2024 · 10d3b80 · 10d3b80
2 parents db2a026 + 7d5bc4a
commit 10d3b80
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 4 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -14,7 +14,7 @@ members = [
 ]
 
 [workspace.dependencies.zarrs_metadata]
-version = "0.1.0"
+version = "0.2.0"
 path = "zarrs_metadata"
 
 [workspace.dependencies.zarrs_storage]

diff --git a/zarrs/Cargo.toml b/zarrs/Cargo.toml
@@ -58,7 +58,7 @@ num = { version = "0.4.1" }
 pco = { version = "0.3.1", optional = true }
 rayon = "1.10.0"
 rayon_iter_concurrent_limit = "0.2.0"
-serde = { version = "1.0.184", features = ["derive"] }
+serde = { version = "1.0.185", features = ["derive"] }
 serde_json = { version = "1.0.71", features = ["float_roundtrip", "preserve_order"] }
 thiserror = "1.0.61"
 thread_local = "1.1.8"

diff --git a/zarrs_metadata/CHANGELOG.md b/zarrs_metadata/CHANGELOG.md
@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+ - **Breaking**: Mark `GroupMetadataV3` and `ArrayMetadataV3` as non-exhaustive
+
 ## [0.1.0] - 2024-09-02
 
 ### Added

diff --git a/zarrs_metadata/Cargo.toml b/zarrs_metadata/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "zarrs_metadata"
-version = "0.1.0"
+version = "0.2.0"
 authors = ["Lachlan Deakin <[email protected]>"]
 edition = "2021"
 rust-version = "1.76"
@@ -16,7 +16,7 @@ derive_more = { version = "1.0.0", features = ["display", "from"] }
 half = { version = "2.0.0", features = ["bytemuck"] }
 monostate = "0.1.0"
 num = { version = "0.4.1" }
-serde = { version = "1.0.184", features = ["derive"] }
+serde = { version = "1.0.185", features = ["derive"] }
 serde_json = { version = "1.0.71", features = ["float_roundtrip", "preserve_order"] }
 serde_repr = "0.1.19"
 thiserror = "1.0.61"
diff --git a/zarrs_metadata/src/v3/array.rs b/zarrs_metadata/src/v3/array.rs
@@ -97,6 +97,7 @@ pub mod nan_representations;
 ///     }
 /// }
 /// ```
+#[non_exhaustive]
 #[derive(Serialize, Deserialize, Clone, PartialEq, Debug, Display)]
 #[display("{}", serde_json::to_string(self).unwrap_or_default())]
 pub struct ArrayMetadataV3 {

diff --git a/zarrs_metadata/src/v3/group.rs b/zarrs_metadata/src/v3/group.rs
@@ -17,6 +17,7 @@ use super::AdditionalFields;
 ///         "eggs": 42,
 ///     }
 /// }
+#[non_exhaustive]
 #[derive(Serialize, Deserialize, Clone, Eq, PartialEq, Debug, Display)]
 #[display("{}", serde_json::to_string(self).unwrap_or_default())]
 pub struct GroupMetadataV3 {