Skip to content

Migration from yaml-rust2 to saphyr? #654

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

Open
LecrisUT opened this issue Apr 5, 2025 · 4 comments · May be fixed by #656
Open

Migration from yaml-rust2 to saphyr? #654

LecrisUT opened this issue Apr 5, 2025 · 4 comments · May be fixed by #656

Comments

@LecrisUT
Copy link

LecrisUT commented Apr 5, 2025

They yaml-rust2 crate is indicating that it is in maintenance mode 1 and it is recommending migration to saphyr. The issue mentions that these are fairly similar, but I'm not sure if it refers to the API being similar.

Footnotes

  1. https://github.com/Ethiraric/yaml-rust2/issues/26

@LecrisUT LecrisUT linked a pull request Apr 9, 2025 that will close this issue
@epage
Copy link
Contributor

epage commented Apr 9, 2025

Thanks for bringing saphyr to our attention!

it is recommending migration to saphyr.

Maybe I missed something in my read but I didn't quite read it that way. yaml-rust2 is still supported. The distinction mostly comes down to what your tolerance for breaking changes / churn is. The fact that the version is only 0.0.3 communicates a level of instability that I would hesitate to take on at this time. I also would worry for the quality perception people would have in seeing a 0.0.x dependency in their dependency tree.

For now, I lean towards sticking with yaml-rust2 and that we can re-evaluate that as saphyr matures.

@LecrisUT
Copy link
Author

My main concern is if yaml-rust2 will get dependencies updated in time. So far neither of them have a dependabot/renovatebot setup. If they plan to keep them both updated, then yeah it would make sense to pick the more stable one for now. I have tried to contribute a renovatebot setup, but I couldn't cook up a minimal one to propose. Maybe you guys have more experience and could make a PR for that?

@epage
Copy link
Contributor

epage commented Apr 10, 2025

The setup I use for all of my libraries is at https://github.com/epage/_rust/blob/main/.github/renovate.json5

@LecrisUT
Copy link
Author

I thought that one had a required setup of labels and such. But yeah, it could work. I'll make a few PRs for those

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants