-
Notifications
You must be signed in to change notification settings - Fork 508
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
Unit tests required - Vulnerabilities check #1535
Comments
@laurentsimon FYI |
the same tests we had before are there. It should cover the same logic, maybe percentage-wise it's down because the checks/vulnerabilities.go introduced a new level of indirection/boilerplate. Which lines are not hit that use to be? |
where do we see the coverage for a PR? I did not notice it in the main thread. Is it in a check run's details? |
I know, I wish the tool was reported in the PR. I don't know why it was missing TBH. |
Stale issue message |
Needs more tests for this https://codecov.io/gh/ossf/scorecard/commit/5f9fff3b20ce7eb933978c7a4f9391cb2c9b3d89
Originally posted by @naveensrinivasan in #1532 (comment)
The text was updated successfully, but these errors were encountered: