-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Slim Dependencies #2691
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
Comments
I thought I was dreaming when I saw this issue.
|
ah, it sucks that all of these are transitive dependencies and that we don't really have much control over it. |
Can I just say that @wtgtybhertgeghgtwtg's analysis is incredibly thorough. Thank you for doing that |
Yep, this is really great work - thanks @wtgtybhertgeghgtwtg |
note-notifier dropped the cli: #2718 |
Nice! |
Just an overview of the impact this has had. |
Nice work, thanks @wtgtybhertgeghgtwtg for working on this and getting back with more data :) |
+1 |
I think we are mostly good on this for now and we sped Jest's startup up by more than 50% with Jest 21. If anybody wants to drop more stuff, please feel free to comment here and send PRs! |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Also see facebook/create-react-app#1183
We should focus on reducing the amount of dependencies that are shipped with Jest. Is there somebody that could do an analysis?
The text was updated successfully, but these errors were encountered: