-
Notifications
You must be signed in to change notification settings - Fork 149
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
test express dependency tree #631
Comments
Current list of express' direct dependencies: { accepts: '~1.3.5',
'array-flatten': '1.1.1',
'body-parser': '1.18.3',
'content-disposition': '0.5.2',
'content-type': '~1.0.4',
cookie: '0.3.1',
'cookie-signature': '1.0.6',
debug: '2.6.9',
depd: '~1.1.2',
encodeurl: '~1.0.2',
'escape-html': '~1.0.3',
etag: '~1.8.1',
finalhandler: '1.1.1',
fresh: '0.5.2',
'merge-descriptors': '1.0.1',
methods: '~1.1.2',
'on-finished': '~2.3.0',
parseurl: '~1.3.2',
'path-to-regexp': '0.1.7',
'proxy-addr': '~2.0.4',
qs: '6.5.2',
'range-parser': '~1.2.0',
'safe-buffer': '5.1.2',
send: '0.16.2',
'serve-static': '1.13.2',
setprototypeof: '1.1.0',
statuses: '~1.4.0',
'type-is': '~1.6.16',
'utils-merge': '1.0.1',
vary: '~1.1.2' } Currently tested in citgm:
|
I already have an open PR for |
citgm results on all dependencies with Node.js 11.3.0
TLDR
|
I think we should start adding the passing ones immediately, and then add the other ones as soon as they pass. |
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Forked from nodejs/node#24586 (comment).
It seems we are lacking quite a bit of testing.
cc @dougwilson
The text was updated successfully, but these errors were encountered: