-
Notifications
You must be signed in to change notification settings - Fork 113
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
Document snyk usage #3576
Comments
@ascott1 Since we're trialling snyk.io accounts, is having a command in the project necessary? |
I'd lean towards no, assuming we have a process for monitoring/resolving snyk alerts. |
Is this still an issue you want help on? Wasn't sure based on the conversation above. Thanks! |
Hi @saracope, We could use a section in above https://cfpb.github.io/consumerfinance.gov/other-front-end-testing/#performance-testing for "Security testing" that lists running |
Current behavior
snyk
dependency is used for runningnpm run test
, however, this requires an authenticated snyk account. (snyk test requires an authenticated account. Please run
snyk authand try again.
). There is documentation on why snyk is in the project or how it should be setup.Expected behavior
npm run test
is documented in the project testing docs.The text was updated successfully, but these errors were encountered: