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

only load render function, not whole ReactDOM library #146

Closed
wants to merge 1 commit into from
Closed

only load render function, not whole ReactDOM library #146

wants to merge 1 commit into from

Conversation

z11h
Copy link

@z11h z11h commented Jul 23, 2016

No description provided.

@ghost ghost added the CLA Signed label Jul 23, 2016
@gaearon
Copy link
Contributor

gaearon commented Jul 24, 2016

Hi, thanks. This doesn’t actually make any difference in our case (at least, for bundle size). It was like this in the beginning but I changed it to avoid throwing named imports at people in the very first file they see. So I’d prefer to keep it this way for now.

@gaearon gaearon closed this Jul 24, 2016
@z11h z11h deleted the patch-1 branch July 24, 2016 18:10
@lock lock bot locked and limited conversation to collaborators Jan 22, 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.

2 participants