ENT-14036: Fixed a bug with json-parse not failing on missing comma#282
Conversation
|
There was an error running your pipeline, see logs for details. |
larsewi
left a comment
larsewi
left a comment
There was a problem hiding this comment.
Please add some tests and verify that they fail pre fix and succeed post fix
Ticket: ENT-14036 Changelog: None Signed-off-by: Simon Halvorsen <simon.halvorsen@northern.tech>
SimonThalvorsen
force-pushed
the
ENT-14036
branch
from
ab62abd to
57f89b3
Compare
|
There was an error running your pipeline, see logs for details. |
|
@cf-bottom Jenkins please :) |
|
Sure, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/14019/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14019/ |
|
Merge together: |
|
@cf-bottom Jenkins please :) |
|
@SimonThalvorsen : I'm sorry, I cannot do that. @olehermanse please help. |
|
@cf-bottom jenkins please |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/14025/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14025/ |
|
@cf-bottom Now try Jenkins please :) |
|
@SimonThalvorsen : I'm sorry, I cannot do that. @olehermanse please help. |
|
@larsewi please help |
|
@cf-bottom jenkins please |
|
Alright, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/14031/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14031/ |
|
@cf-bottom Jenkins :) |
|
Sure, I triggered a build: Jenkins: https://ci.cfengine.com/job/pr-pipeline/14040/ Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14040/ |
5 participants
Ticket: ENT-14036
Changelog: None
Merge together:
cfengine/core#6195
https://github.com/cfengine/nova/pull/2637