-
Notifications
You must be signed in to change notification settings - Fork 99
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
feat(cve): cli cve diff #2242
feat(cve): cli cve diff #2242
Conversation
4f2dd6a
to
f90acff
Compare
f90acff
to
233c622
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2242 +/- ##
==========================================
+ Coverage 92.81% 92.82% +0.01%
==========================================
Files 167 167
Lines 21855 22250 +395
==========================================
+ Hits 20285 20654 +369
- Misses 984 996 +12
- Partials 586 600 +14 ☔ View full report in Codecov by Sentry. |
65c2532
to
497dd96
Compare
497dd96
to
5c57763
Compare
Example: Should print CVE's in python:3.8.16-alpine3.17 and not in python:3.8.16-alpine3.16 |
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.
Except the rename it looks good. Please add more tests to increase coverage
ec631d8
to
4451aae
Compare
4451aae
to
51d240f
Compare
Signed-off-by: Laurentiu Niculae <niculae.laurentiu1@gmail.com>
0183249
to
b437bc0
Compare
Signed-off-by: Laurentiu Niculae <niculae.laurentiu1@gmail.com>
b437bc0
to
14884da
Compare
What type of PR is this?
Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:
Automation added to e2e:
Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.