Skip to content

Output the monitor result as a json#381

Merged
adrukh merged 1 commit intosnyk:fix/monitor-result-in-dockerfrom
julienduchesne:output-monitor-result
Feb 27, 2019
Merged

Output the monitor result as a json#381
adrukh merged 1 commit intosnyk:fix/monitor-result-in-dockerfrom
julienduchesne:output-monitor-result

Conversation

@julienduchesne
Copy link
Contributor

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Currently, only the regular result is output
However, the monitor result has some useful info such as the scan URL

Screenshots

image

Currently, only the regular result is output
However, the monitor result has some useful info such as the scan URL
@julienduchesne
Copy link
Contributor Author

Replaces part of #164

@adrukh adrukh changed the base branch from master to fix/monitor-result-in-docker February 27, 2019 14:30
@adrukh
Copy link
Contributor

adrukh commented Feb 27, 2019

Merging to a side branch to overcome CI limitation with forked PRs. Will merge this soon, thanks @julienduchesne !

@adrukh adrukh merged commit a45fae4 into snyk:fix/monitor-result-in-docker Feb 27, 2019
@julienduchesne julienduchesne deleted the output-monitor-result branch February 27, 2019 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants