This repository has been archived by the owner on Sep 28, 2022. It is now read-only.
Switch TOML implementation to toml_edit #492
Comments
sunshowers
changed the title
sunshowers
added a commit
to sunshowers/cargo-guppy
that referenced
this issue
Nov 20, 2021
This means that the same library can parse summaries created by both old and new versions of guppy. We don't use the summary for much anyway, currently. I also looked at switching to `toml_edit`, which has a much better design overall. Unfortunately, I ran into toml-rs/toml#192. It makes sense to switch to `toml_edit` at some point in the future, and I've filed facebookarchive#492 to keep track of that.
sunshowers
added a commit
to sunshowers/cargo-guppy
that referenced
this issue
Nov 23, 2021
This means that the same library can parse summaries created by both old and new versions of guppy. We don't use the summary for much anyway, currently. I also looked at switching to `toml_edit`, which has a much better design overall. Unfortunately, I ran into toml-rs/toml#192. It makes sense to switch to `toml_edit` at some point in the future, and I've filed facebookarchive#492 to keep track of that.
sunshowers
added a commit
that referenced
this issue
Nov 23, 2021
This means that the same library can parse summaries created by both old and new versions of guppy. We don't use the summary for much anyway, currently. I also looked at switching to `toml_edit`, which has a much better design overall. Unfortunately, I ran into toml-rs/toml#192. It makes sense to switch to `toml_edit` at some point in the future, and I've filed #492 to keep track of that.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
toml_editlooks like the future of toml parsing and serialization in Rust: its design avoidsValueAfterTableissues, provides greater control over formatting if necessary, etc. I think it makes sense to switch to it at some point. Filing this task to keep track of blocking issues.easy::to_string_prettydoesn't produce pretty results toml-rs/toml#192The text was updated successfully, but these errors were encountered: