-
Notifications
You must be signed in to change notification settings - Fork 223
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
CVE applicability indication for jf audit
#1939
Conversation
orto17
commented
May 8, 2023
- All tests passed. If this feature is not already covered by the tests, I added new tests.
- All static analysis checks passed.
- This pull request is on the dev branch.
- I used gofmt for formatting the code before submitting the pull request.
jf audit
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.
Go ahead and add the new functionality to the JFrog CLI documentation. The source files for the documentation and managed under the "documentation" directory.
In addition to the description for the new Contextual Analysis feature, let's also explain how to subscribe to the new Advanced Security package.
I also suggest adding a few images to showcase the new functionality as part of the "jf audit" command.
Let's also ensure that all the tests are passing.
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.
@orto17,
I have some comments for the documentation, but I suggest we merge this PR, and then I'll create a follow-up PR to improve the documentation.
Please go ahead and resolve the conflicts, and we'll merge once the tests are grenn.
# Conflicts: # go.mod # go.sum