Skip to content
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(scanner): slight update in version display #3063

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

ffontaine
Copy link
Contributor

Replace v{product_info.version} by version {product_info.version} to allow the version number to be displayed in the same color (i.e. 4.19.205 will be displayed in blue and version in white instead of displaying 19.205 in blue and v4. in white)

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.

Nice thinking. Looks like it breaks one test, though:

FAILED test/test_csv2cve.py::TestCSV2CVE::test_csv2cve_valid_file - assert 0 >= 60

I'm actually kind of surprised it was only the one that broke.

Replace "v{product_info.version}" by "version {product_info.version}" to
allow the version number to be displayed in the same color (i.e.
"4.19.205" will be displayed in blue and "version" in white instead of
displaying "19.205" in blue and "v4." in white)

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
@codecov-commenter
Copy link

Codecov Report

Merging #3063 (a73b886) into main (6e807b1) will decrease coverage by 6.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3063      +/-   ##
==========================================
- Coverage   82.92%   76.02%   -6.90%     
==========================================
  Files         674      702      +28     
  Lines       10650    10923     +273     
  Branches     1429     1476      +47     
==========================================
- Hits         8831     8304     -527     
- Misses       1457     2292     +835     
+ Partials      362      327      -35     
Flag Coverage Δ
longtests 76.02% <ø> (-6.43%) ⬇️
win-longtests ?

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

Impacted Files Coverage Δ
cve_bin_tool/cve_scanner.py 83.72% <ø> (ø)
test/test_csv2cve.py 100.00% <ø> (ø)

... and 65 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

Time to merge. The test timeouts don't look concerning, and I've upped those limits in main today so hopefully we won't see them happening as frequently in future. Thank you!

@terriko terriko merged commit 3c3cd2b into intel:main Jun 6, 2023
19 of 22 checks passed
@ffontaine ffontaine deleted the slight-update branch June 6, 2023 20:53
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.

None yet

3 participants