Remove serde yaml, update some deps to the latest version.#181
Merged
TheJokr merged 8 commits intocloudflare:mainfrom Mar 26, 2026
Merged
Remove serde yaml, update some deps to the latest version.#181TheJokr merged 8 commits intocloudflare:mainfrom
TheJokr merged 8 commits intocloudflare:mainfrom
Conversation
Collaborator
|
Getting rid of serde-yaml would be nice. I added some commits to retain I also reverted the dependency bumps to keep the PR focused. There's no need to bump minor versions in a library. |
266ad0e to
b9da2c8
Compare
b9da2c8 to
74850a9
Compare
TheJokr
approved these changes
Mar 26, 2026
fisherdarling
approved these changes
Mar 26, 2026
Collaborator
|
Thank you for the contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
serde_yamlwas archived on March 25, 2024 and is no longer maintained,generating CVE warnings for downstream consumers of this library. This is
a blocker for anyone depending on
foundationswho runscargo auditorsimilar auditing tools.
serde_yamlwithserde-saphyr, the actively maintainedsuccessor backed by the
saphyrYAML parseryaml-merge-keysentirely —serde-saphyrhandles YAML mergekeys (
<<:) natively and transparently during deserialization, makingthe explicit merge pass redundant
from_yaml_strfrom a three-stepparse →
Value→ merge → deserialize pipeline down to a singleserde_saphyr::from_strcallversions (
tokio,regex,thiserror,pin-project-lite, etc.)No behaviour change. YAML merge key semantics are preserved — the
processing is now done inside
serde-saphyrrather than as a manualpost-processing step.
Existing test suite passes unchanged. No new tests required as this is
a pure dependency substitution with equivalent semantics.