You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Now that we have a monorepo structure, we can pull out some of the small and annoying dependencies from packages/react-scripts/package.json into packages/react-dev-utils/package.json and provide helpers there instead.
I started this in #723 but packages/react-scripts/scripts/build.js is still pretty noisy, and we could extract the code to compare build sizes (and maybe something else) into react-dev-utils.
I recommend the following workflow:
Look at packages/react-scripts/package.json
See where these dependencies get used
If they seem non-essential and too low-level, figure out how to hide them in react-dev-utils
Repeat
This is a contributor issue. I don’t plan to work on this right now so please write a comment if you intend to take it. Please don’t start working without a comment so we don’t have many people solving the same problem.
If you need any help, ping me in this issue and I’ll be happy to reply or explain something.
I think they’re technically useful, it’s just that we don’t want them directly under react-scripts because its deps pollute the project after ejecting. We now have a utility module (react-dev-utils), and we should just extract things users won’t want to change there.
Activity
dalamgir commentedon Sep 25, 2016
will take a crack at it
gaearon commentedon Sep 25, 2016
You got it @Alamgird!
amilajack commentedon Sep 25, 2016
npm-check might be a good solution for this
gaearon commentedon Sep 25, 2016
I think they’re technically useful, it’s just that we don’t want them directly under
react-scriptsbecause its deps pollute the project after ejecting. We now have a utility module (react-dev-utils), and we should just extract things users won’t want to change there.Timer commentedon Mar 23, 2017
@Alamgird I'm going to open this back up unless if you are still interested in working on it.
Remove unnecessary dependencies(#751) (#2283)
Remove connect-history-api-fallback and cross-spawn dependencies from…
Remove unnecessary dependencies(react#751) (react#2283)
Remove connect-history-api-fallback and cross-spawn dependencies from…
Remove connect-history-api-fallback and cross-spawn dependencies from…
Remove unnecessary dependencies(react#751) (react#2283)
Remove connect-history-api-fallback and cross-spawn dependencies from…
Merge upstream create-react-app v1.0.7 (#90)
ppatel221 commentedon Dec 7, 2017
Hey if this is still open, I'd like to work on it!
Timer commentedon Dec 7, 2017
I think some progress was made, but there's always more potential @ppatel221. Feel free to look into it. 😄
gaearon commentedon Jan 8, 2018
I'll close as stale.
Move chalk dependency to react-dev-utils (react#6150)