-
Notifications
You must be signed in to change notification settings - Fork 43
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
Long summary output for only one vulnerable advisory #239
Comments
The duplications are the concern. I must have missed a deduplication spot (I have them in a few places). IMO, that's the fix required in this scenario. Thanks for reporting! |
Closed with #240, will create a release shortly. |
Released with |
@quinnturner Thanks for the quick change. Unfortunately the summary still has the same lines as I created a repo which shows this: Here the found paths output as well:
|
Fix released in |
Just ran
Thanks @kyletsang |
We have a warning for this advisory in our system:
GHSA-xvch-5gv4-984h
But the summary output of the audit-ci run does not look good in my opinion:
Of course we just need to upgrade
minimist
and the output goes away, but I do think that the summary should be displayed differently here.I will try to find time to create a dummy repo for this, but for now here the needed
yarn.lock
andaudit-config.json
content.audit-config.json
The text was updated successfully, but these errors were encountered: