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

fix: update detailed description #3650

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

ffontaine
Copy link
Contributor

detailed option was added by commit 4f3538e with the following description: "display detailed report" which is a bit cryptic so replace it by "add CVE description in csv or json report (no effect on console, html or pdf)".

detailed option was added by commit
4f3538e with the following description:
"display detailed report" which is a bit cryptic so replace it by "add
CVE description in csv or json report (no effect on console, html or
pdf)".

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
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.

This is a lot clearer, thank you.

@terriko
Copy link
Contributor

terriko commented Dec 20, 2023

Also, the test fails in windows are clearly unrelated. I'm investigating them elsewhere so I might wait on merging for an hour or so while those tests run and I debug.

@terriko
Copy link
Contributor

terriko commented Dec 20, 2023

Debug tests done, going to get this merged now. thank you for your patience!

@terriko terriko merged commit 10654c4 into intel:main Dec 20, 2023
19 of 21 checks passed
@ffontaine ffontaine deleted the fix-detailed-description branch December 21, 2023 13:46
inosmeet pushed a commit to inosmeet/cve-bin-tool that referenced this pull request Feb 16, 2024
detailed option was added by commit
4f3538e with the following description:
"display detailed report" which is a bit cryptic so replace it by "add
CVE description in csv or json report (no effect on console, html or
pdf)".

Signed-off-by: Fabrice Fontaine <fabrice.fontaine@orange.com>
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.

2 participants