Skip to content
This repository has been archived by the owner on Sep 5, 2019. It is now read-only.

Added 2 new configuration options 'failOnVulns' and 'outputFile' #20

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

Conversation

grantjforrester
Copy link

We want to use the grunt-nsp task as part of a larger build process and would like it to output in a way that can be picked up by other tools (i.e. a file) and not fail the build so other tasks can run.

This pull request adds these two simple options which are both optional so existing users will be unaffected by the change.

@nlf
Copy link
Member

nlf commented Jul 8, 2016

i think the outputFile option belongs in nsp itself rather than here.

the failOnVulns option is an interesting idea, but how you have it implemented here looks like if failOnVulns is false and outputFile is not set, the results of the check are never displayed anywhere

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants