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

Take into account severity #7

Merged
merged 2 commits into from
Jul 9, 2020
Merged

Take into account severity #7

merged 2 commits into from
Jul 9, 2020

Conversation

IPWright83
Copy link
Contributor

@IPWright83 IPWright83 commented Jul 8, 2020

This should allow for #4

Sorry it's a bit of a hefty change, to instead use JSON as it's much easier to deal with from a processing point of view. Also I forgot to disable my prettier extension first :/

@jeemok jeemok merged commit 70da2ed into jeemok:master Jul 9, 2020
@IPWright83
Copy link
Contributor Author

@jeemok thanks for the shout out :)

Honestly we've been looking at npm audit and how we can bake it into our continuous integration. When I discovered that I couldn't ignore vulnerabilities I came across this. Just needed a little more work to get it suitable for our use :)

@jeemok
Copy link
Owner

jeemok commented Jul 9, 2020

No worries, thank you for improving it! :) I'm glad that this repository is helping you and your team!

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.

2 participants