feat: config tree size limit for yarn and npm.

anthogez commented 4 years ago

[x] Tests written and linted
[x] Documentation written / README.md updated https://snyk.io/docs/snyk-for-node/
[x] Follows CONTRIBUTING agreement
[x] Commit history is tidy https://git-scm.com/book/en/v2/Git-Branching-Rebasing
[x] Reviewed by Snyk team

config treeSizeLimit for yarn & npm

This will be improved in another pr with configurable treeSizeLimit tests for yarn & npm

Edit -- Side pr no longer necessary because the side pr upgrade are already applied on master so we could proceed with this

anthogez commented 4 years ago

can we add a test to assert a TreeSizeImitError?

Now that snyk-config was implemented and we will no longer need a side pr, now we have as well the tests.

anthogez commented 4 years ago

Looks good to me, can there be any tests in place for that?

Were already added :)

snyksec commented 4 years ago

:tada: This PR is included in version 1.20.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

snyk / nodejs-lockfile-parser