Skip to content

fix: Deprecate NVD API 1.0 #3671

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

Merged
merged 5 commits into from
Jan 3, 2024
Merged

fix: Deprecate NVD API 1.0 #3671

merged 5 commits into from
Jan 3, 2024

Conversation

akshatgokul
Copy link
Contributor

@akshatgokul
Copy link
Contributor Author

Hi @terriko, can you please review this? Thanks!

@terriko
Copy link
Contributor

terriko commented Dec 27, 2023

I've approved the workflow to run tests. I'll be back to review more carefully sometime after the tests all pass (but we're having some CI issues at the moment so that may not happen today).

@akshatgokul
Copy link
Contributor Author

I have fixed a small linting problem, and apparently 'failover' is not a recognized word by the dictionaries.

@codecov-commenter
Copy link

codecov-commenter commented Dec 28, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (bf9afe9) 78.98% compared to head (332a117) 79.24%.

Files Patch % Lines
cve_bin_tool/cli.py 0.00% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3671      +/-   ##
==========================================
+ Coverage   78.98%   79.24%   +0.26%     
==========================================
  Files         797      797              
  Lines       11927    11930       +3     
  Branches     1602     1603       +1     
==========================================
+ Hits         9420     9454      +34     
+ Misses       2075     2041      -34     
- Partials      432      435       +3     
Flag Coverage Δ
longtests 78.71% <0.00%> (+0.40%) ⬆️
win-longtests 77.11% <0.00%> (+0.26%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@akshatgokul akshatgokul requested a review from terriko January 1, 2024 17:45
Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we're all ready to merge now. Thank you and congratulations on your first merged commit with us!

@terriko terriko merged commit 1aeb605 into intel:main Jan 3, 2024
inosmeet pushed a commit to inosmeet/cve-bin-tool that referenced this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: Deprecate NVD API 1.0
3 participants