-
Notifications
You must be signed in to change notification settings - Fork 23
CFE-4616: Fix control chars escaping in JSON serialization #260
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
Conversation
|
There was an error running your pipeline, see logs for details. |
larsewi
left a comment
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.
Thanks! I have some smaller comments on code style in regards to the consistency with the rest of the code base (see CONTRIBUTING.md). We can fix it for you if you want. Great work 🚀
6468311 to
065c8ee
Compare
|
There was an error running your pipeline, see logs for details. |
Ticket: CFE-4616
065c8ee to
6d4db47
Compare
|
There was an error running your pipeline, see logs for details. |
|
@cf-bottom Jenkins please :) |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/13117/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-13117/ |
No description provided.