Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Move nsp check from prepare to an optional script #15

Closed
lholmquist opened this issue Feb 20, 2018 · 0 comments · Fixed by #16
Closed

Move nsp check from prepare to an optional script #15

lholmquist opened this issue Feb 20, 2018 · 0 comments · Fixed by #16
Assignees
@lholmquist
Labels
enhancement New feature or request

Comments

@lholmquist
Copy link
Contributor

Instead, we should just echo a message in prepare. E.g.

"prepare": "echo 'To confirm CVE compliance, run 'npm run security-check'",
"security-check: nsp check"

or something...
@lholmquist lholmquist self-assigned this Feb 20, 2018
@lholmquist lholmquist added the enhancement New feature or request label Feb 20, 2018
lholmquist added a commit to lholmquist/nodejs-rest-http-secured-redhat that referenced this issue Feb 20, 2018
@lholmquist
feat(package): Move nsp check from prepare step to an optional script.
decf1a2 
* prepare will now output a message telling the user how to run nsp.

fixes nodeshift-archived#15
@lholmquist lholmquist mentioned this issue Feb 20, 2018
Merged
@ghost ghost added the in progress label Feb 20, 2018
lholmquist added a commit that referenced this issue Feb 20, 2018
@lholmquist
feat(package): Move nsp check from prepare step to an optional script. (
be99104 
#16)

* prepare will now output a message telling the user how to run nsp.

fixes #15
@ghost ghost removed the in progress label Feb 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lholmquist lholmquist

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
1 participant
@lholmquist