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

Add npm audit v2 parsing #9

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add npm audit v2 parsing #9

wants to merge 8 commits into from

Conversation

elpete
Copy link

@elpete elpete commented May 6, 2021

This PR automatically detects the version of the audit json coming in and parses it accordingly.

This has also been published to NPM as @elpete/gitlab-npm-audit-parser.

Resolves #8

elpete and others added 8 commits May 6, 2021 10:03
### Rationale
Automates the testcases using the npm test infrastructure
(`npm test`).  Adds the actual expected outcomes as defined files which
are verified to be the same
Also ensures the output file has a EOF character at end.
### Rationale
Tests were not providing a bad exit code & the latest npm v7.13.0, also
no longer throws an abnoxous error on a failed run-scripts.

This should be a bit easier to determine when testing.
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.

Missing support for NPM audit v2
2 participants