-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[3006.x] fix yaml output #66783
[3006.x] fix yaml output #66783
Conversation
@tomdoherty This might be better done on the 3006.x branch, given it is a 3006 bug, and the fix will see release sooner that any fix from the master branch. Fixes on 3006.x are merge-forwarded to 3007.x and master branches. |
01b3c5e
to
afebba3
Compare
Thanks for the heads-up @dmurphy18! Done |
In b9be2de, OrderedDict was replaced with HashableOrderedDict. Add logic to yamldumper to handle the new type.
2fe075b
to
b7b29ec
Compare
Congratulations on your first PR being merged! 🎉 |
In b9be2de, OrderedDict was replaced with HashableOrderedDict. Add logic to yamldumper to handle the new type.
What does this PR do?
What issues does this PR fix or reference?
Fixes #66594
Previous Behavior
New Behavior
Merge requirements satisfied?
[NOTICE] Bug fixes or features added to Salt require tests.
Commits signed with GPG?
Yes/No
Please review Salt's Contributing Guide for best practices, including the
PR Guidelines.
See GitHub's page on GPG signing for more information about signing commits with GPG.