-
Notifications
You must be signed in to change notification settings - Fork 177
Conversation
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks! If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions. |
seems like this should be merged / released to npm? |
There must be a problem with @facebook-github-bot , because I did sign that CLA. Anyway, I won't be upset if someone else makes the change, let's make it happen! |
i also had to change this line:
to this:
maybe that is a difference in a newer version of react or something. I ended up just cloning the repo and poking it until it started working : p idk anything about react internals |
Yeah, I have a note about that in my first comment |
ah yeah missed that :) |
Until this is merged, it's easy enough to patch react-art with a postinstall script in your project's
|
Thanks @quinn .I have solved my problem. |
Well cool, it's done and in master now 🎉 |
Sorry we didn't get this merged. Thanks for sending! |
As pointed in facebook/react#7770 (comment), some internal modules of React 15.4.0 will be moved to react-dom package. When this happens, React ART will have to require
ReactInstanceMap
,ReactMultiChild
,ReactUpdates
from react-dom instead of react.Fixes #103.
Note for maintainers: Before merging this, you might want to revert db4dcef, since it's purpose is delayed until React 16 (facebook/react#7598). This is the reason why tests are failing right now.
💃