Skip to content
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

Revert "Add React Styleguidist" #2646

Closed
wants to merge 2 commits into from
Closed

Revert "Add React Styleguidist" #2646

wants to merge 2 commits into from

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