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
Fix TOML serialize for package default_cfg #1914
Conversation
Signed-off-by: Salim Alam <salam@chef.io>
Thanks for the pull request! Here is what will happen next:
Thank you for contributing! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thesentinels approve |
🤘 I am testing your branch against master before merging it. We do this to ensure that the master branch is never failing tests. |
Travis CI has started testing this PR. |
💖 Travis CI reports this PR passed. It always makes me feel nice when humans approve of one anothers work. I'm merging this PR now. I just want you and the contributor to answer me one question: |
Approved by: @nobody from Nowhere Merged by: The Sentinels
This fixes the breakage described in issue #1908. The package default config is now deserialized as a TOML Value, instead of the a TOML Table, which preserves the ordering constraint that TOML serialization requires.
Signed-off-by: Salim Alam salam@chef.io