Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
0.9
refactored theNumber
value variant to be lossless, which requires more thorough matching.Cargo.toml
, (asValueKind
supports an equivalent type,config-rs
could opt-in).0.9
also introduces a newBytes
value variant to support rusty byte strings (Vec<u8>
). This was to resolve a roundtrip serialization bug with base64 strings when serdedeserialize_any
is used (which lacks a type hint and would deserialize as a literal string to bytes, instead of base64 decoded string to bytes).Relevance
I've had this sitting around for a while and meaning to push a PR before I forget 😅
This PR is an alternative to #472 and a sibling of #474
In particular Ron
0.9
is intended to be more robust withValue
better supporting untagged enums and similar compatibility issues.from_value
method #472 , but forconfig-rs
is passing the Ron tests successfully regardless of Ron0.8
or0.9
release used.0.9
support there asdeserialize_any
to the untagged enum will fallback to theNil
variant when a variant is unsupported byconfig-rs
.config-rs
is generic across formats, that's less important so long as functionally the format meets test expectations.Ron
0.10
will continue to improve on theValue
type, which may be relevant forconfig-rs
.