search

sonatype-nexus-community / jake

Check your Python environments for vulnerable Open Source packages with OSS Index or Sonatype Nexus Lifecycle.
https://jake.readthedocs.io/
Apache License 2.0
111 stars 24 forks source link

Add json output format for ddt command #56

Closed therealak12 closed 3 years ago

therealak12 commented 3 years ago

Currently ddt can only print output in human readable format. This PR adds the option to print found vulnerabilities only in JSON format.

This pull request makes the following changes:

cc @bhamail / @DarthHater

bhamail commented 3 years ago

@therealak12 Thanks for this PR. I noticed our CI was not configured to build from forks. I've fixed that, but then noticed the build failing for this PR. Could you take a look and see if you can fix that failure?

FWIW, you can likely reproduce the CI error locally by following these steps: CI Debug Notes

I was able to reproduce the error locally using these steps.

therealak12 commented 3 years ago

@bhamail I fixed the CI errors. Would you please check again?

therealak12 commented 3 years ago

@bhamail You may need to uncheck ci/circleci. See here.

bhamail commented 3 years ago

@therealak12 It looks like CI is behaving now. Might need to resolve the merge conflicts GH is flagging.

bhamail commented 3 years ago

@therealak12 Could you merge the recent changes from main into this branch/PR? I'm guessing the branch conflicts will be resolved by doing so.

After that, looks good to go.

therealak12 commented 3 years ago

Sure I will.