Skip to content

Commit

Permalink
revise UnsupportedAdditionalFieldError future change notes
Browse files Browse the repository at this point in the history
  • Loading branch information
LDeakin committed Feb 12, 2025
1 parent 14f739b commit aa5ad78
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 4 deletions.
1 change: 0 additions & 1 deletion zarrs/src/array/array_errors.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ pub enum ArrayCreateError {
#[error(transparent)]
NodePathError(#[from] NodePathError),
/// Unsupported additional field.
// FIXME: Remove in 0.4.0
#[error(transparent)]
UnsupportedAdditionalFieldError(#[from] UnsupportedAdditionalFieldError),
/// Unsupported data type.
Expand Down
1 change: 0 additions & 1 deletion zarrs/src/group.rs
Original file line number Diff line number Diff line change
Expand Up @@ -475,7 +475,6 @@ pub enum GroupCreateError {
#[error(transparent)]
NodePathError(#[from] NodePathError),
/// Unsupported additional field.
// FIXME: Remove in 0.4.0
#[error(transparent)]
UnsupportedAdditionalFieldError(UnsupportedAdditionalFieldError),
/// Storage error.
Expand Down
1 change: 0 additions & 1 deletion zarrs_metadata/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed
- Make `AdditionalField` public and permit any JSON type (not just objects)
- `UnsupportedAdditionalFieldError` errors can no longer be triggered and will be removed in the next breaking release

## [0.3.3] - 2025-02-06

Expand Down
2 changes: 1 addition & 1 deletion zarrs_metadata/src/v3/metadata.rs
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ impl ConfigurationInvalidError {
}
}

// FIXME: Remove in 0.4.0
// FIXME: Move to `zarrs` itself in 0.4.0
/// An unsupported additional field error.
///
/// An unsupported field in array or group metadata is an unrecognised field without `"must_understand": false`.
Expand Down

0 comments on commit aa5ad78

Please sign in to comment.