-
Notifications
You must be signed in to change notification settings - Fork 144
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
snyk-sast: added stats for Snyk scans #1633
Conversation
3abcc0f
to
72ce2f8
Compare
I can not change reviewers/asignee |
Me neither. |
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.
Already reviewed at jperezdealgaba#3.
f8f9df7
to
c293a81
Compare
@tnevrlka Would you mind enabling this pipeline please? |
/ok-to-test |
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.
LGTM
@dirgim Thanks for the thumbs up! Can this be merged? |
Solves: https://issues.redhat.com/browse/OSH-769 Adding the stats to snyk scans in the result's SARIF file of successful scans.
c293a81
to
74d6b7b
Compare
/ok-to-test |
Solves: https://issues.redhat.com/browse/OSH-769
Adding the stats to snyk scans in the result's SARIF file of successful scans.
Example of successful pipeline: https://konflux.apps.stone-prod-p02.hjvn.p1.openshiftapps.com/application-pipeline/workspaces/jperezde/applications/test-coverity/pipelineruns/osh-cli-container-konflux-test-2-on-pull-request-znx98
Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.