-
Notifications
You must be signed in to change notification settings - Fork 221
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove YAML support #473
Comments
It might be unmaintained, but as long as it's working and not causing any problems in There's an actively developed serde yaml crate, although it's unclear if it's at parity feature wise. |
Ah yeah, I didn't see that. I would prefer switching to another YAML implementation rather than removing support completely. |
Just a heads-up, not sure if you get notifications for new PRs but #474 has been open without any engagement nearing two weeks now.
If you merge #474 first, I'll rebase #472 to accomodate, otherwise #472 has a separate commit focused on the YAML support and switch, which should be fine as you prefer merge commits over squash commits, there is little advantage then from merging #474 . Both PRs cross-reference each other, so that information on #474 shouldn't be difficult to find (as those additional details aren't in the commit message). |
Apparently there's the yaml-rust2 drop-in replacement that is actively maintained. I've created #554 with this change to fix #553 which arose in the meantime. |
Originally posted by @polarathene in #466 (comment)
I know that YAML is widely used as a configuration language in some ecosystems, so I am a bit undecided about this one. If someone wants to go ahead and remove YAML support, though, I would merge such a patchset.
The text was updated successfully, but these errors were encountered: