Closure Compiler wrong assignment order in `instantiateReactComponent` by anmonteiro · Pull Request #8895 · react/react · GitHub
Skip to content

Closure Compiler wrong assignment order in instantiateReactComponent#8895

Merged
gaearon merged 1 commit into
react:masterfrom
anmonteiro:react-8894
Feb 6, 2017
Merged

Closure Compiler wrong assignment order in instantiateReactComponent#8895
gaearon merged 1 commit into
react:masterfrom
anmonteiro:react-8894

Conversation

@anmonteiro

@anmonteiro anmonteiro commented Jan 30, 2017

Copy link
Copy Markdown
Contributor

This fixes #8894

@anmonteiro

Copy link
Copy Markdown
Contributor Author

@anmonteiro anmonteiro changed the title Change the order between function declaration and object assignment Closure Compiler wrong assignment order in instantiateReactComponent Feb 5, 2017
@gaearon gaearon added this to the 15-hipri milestone Feb 6, 2017
@gaearon gaearon merged commit 4313059 into react:master Feb 6, 2017
@gaearon

gaearon commented Feb 6, 2017

Copy link
Copy Markdown
Collaborator

This looks fine by itself. However, just to set expectations, I’m not quite sure when we’ll cut another patch release, and React 16 will contain a completely rewritten core, so this change won’t affect it.

@anmonteiro anmonteiro deleted the react-8894 branch February 6, 2017 16:36
@anmonteiro

Copy link
Copy Markdown
Contributor Author

Thanks!

I understand. So to avoid encountering similar problems with Fiber, are there instructions to build master locally and take it for a spin?

@gaearon

gaearon commented Feb 6, 2017

Copy link
Copy Markdown
Collaborator

@anmonteiro

Yea. You should be able to build from master and then import react-dom/fiber instead of react-dom.

@anmonteiro

Copy link
Copy Markdown
Contributor Author

Great. Will do and report any issues I encounter along the way.

@anmonteiro

Copy link
Copy Markdown
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Closure Compiler: wrong assignment order in instantiateReactComponent

3 participants