Skip to content
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

replace unmaintained yaml-rust with yaml-rust2 #19

Merged

Conversation

NathanReus
Copy link
Contributor

@NathanReus NathanReus commented May 24, 2024

yaml-rust appears to be unmaintained however, the community has published an actively maintained fork under the name of yaml-rust2 (package, repo). It was forked from the latest commit of yaml-rust and is a drop-in replacement, as well as being improved by multiple developers (details of which are in the linked issue above).

RustSec advisory advising this course of action can be found here: RUSTSEC-2024-0320.

Copy link
Collaborator

@yuchanns yuchanns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on green. Thank you!

@yuchanns yuchanns merged commit fe8c121 into the-alchemists-of-arland:main May 24, 2024
9 checks passed
@NathanReus
Copy link
Contributor Author

LGTM on green. Thank you!

No problem! Really glad to be able to help actually, it's my first ever PR contribution to any open source project 😄

@NathanReus NathanReus deleted the yaml-rust-unmaintained branch May 26, 2024 12:50
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 this pull request may close these issues.

2 participants