Skip to content

Conversation

gaearon
Copy link
Contributor

@gaearon gaearon commented Jun 27, 2017

Reverts #2044

Not merging this yet, but adding webpack to explicit dependencies of project is a showstopper for us.

See my comment in https://github.com/facebookincubator/create-react-app/pull/2044/files#r124402656. I'm sorry I didn't notice this before.

If it's possible to avoid this I'd love to know—otherwise I don't think we can keep this suggestion in the docs because it burned many people before.

@gaearon
Copy link
Contributor Author

gaearon commented Jun 27, 2017

cc @sapegin

@gaearon gaearon added this to the 1.0.8 milestone Jun 27, 2017
@sapegin
Copy link
Contributor

sapegin commented Jun 28, 2017

Answered in the initial PR :-) I think we can avoid that.

@gaearon
Copy link
Contributor Author

gaearon commented Jun 28, 2017

See #2656.

@gaearon gaearon closed this Jun 28, 2017
@gaearon gaearon deleted the revert-2044-patch-1 branch June 28, 2017 11:48
@lock lock bot locked and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants