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

Update 2019.2.0 release notes with other tojson example #51962

Merged
merged 7 commits into from Mar 21, 2019

Conversation

@Ch3LL
Copy link
Contributor

commented Mar 4, 2019

What does this PR do?

Add additional information about the changes to the yaml render. Inspired from this issue: #51925

ping @terminalmage any chance you can review here? I tested the json and yaml filters and it worked so want to double check any reason why these filters would not work.

@Ch3LL Ch3LL requested a review from terminalmage Mar 4, 2019
doc/topics/releases/2019.2.0.rst Outdated Show resolved Hide resolved
Co-Authored-By: Ch3LL <megan.wilhite@gmail.com>
doc/topics/releases/2019.2.0.rst Outdated Show resolved Hide resolved
vutny and others added 4 commits Mar 5, 2019
Co-Authored-By: Ch3LL <megan.wilhite@gmail.com>
@vutny

This comment has been minimized.

Copy link
Contributor

commented Mar 20, 2019

@terminalmage Could you please review this?

@garethgreenaway garethgreenaway self-requested a review Mar 21, 2019
@dwoz
dwoz approved these changes Mar 21, 2019
@Ch3LL Ch3LL merged commit 5387821 into saltstack:2019.2 Mar 21, 2019
4 of 10 checks passed
4 of 10 checks passed
jenkins/pr/py2-centos-7 running py2-centos-7...
Details
jenkins/pr/py2-ubuntu-1604 running py2-ubuntu-1604...
Details
jenkins/pr/py2-windows-2016 running py2-windows-2016...
Details
jenkins/pr/py3-centos-7 running py3-centos-7...
Details
jenkins/pr/py3-ubuntu-1604 running py3-ubuntu-1604...
Details
jenkins/pr/py3-windows-2016 running py3-windows-2016...
Details
WIP Ready for review
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
jenkins/pr/docs The docs job has passed
Details
jenkins/pr/lint Python lint test has passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.