-
Notifications
You must be signed in to change notification settings - Fork 3
docs: revamp vulnerability scanning #70
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
Conversation
docs/user-guide/integrations/vulnerability-scanning/vulnerability-scanning.md
Outdated
Show resolved
Hide resolved
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.
We seem to have done a lot of repetitions and also run a risk of irritating our readers. Same steps, same lines, same wordings in most sections. Things have to be explained only once perfectly and the carry-forward will pass on to other sections. Please consider shortening this PR, @Hii-Arpit.
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.
Suggested some changes at a few place (needs to be incorporated at all other places in all the docs in this PR). We need to do something to fix punctuations and grammar issues too.
docs: grammerly check for clair.md
…tion into scanning-revamp pull
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.
LGTM
Quick Preview
docs: revamp vulnerability scanning