Skip to content

Conversation

@jquense
Copy link
Contributor

@jquense jquense commented Jan 5, 2018

I’m sorry i didn’t include a test, i wasn’t sure where or how to confirm this in one. I’m happy to add it if someone has a sense on where to stick it.

for reference for the change: jquense/react-dom-lite#14 (comment)

gaearon
gaearon previously requested changes Jan 5, 2018
Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is already retrieved here, we should pass it to updateHostComponent instead of reading it again.

@gaearon gaearon dismissed their stale review January 5, 2018 15:35

outdated

Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If CI passes

@jquense jquense merged commit 8d336aa into master Jan 5, 2018
@jquense jquense deleted the reconciler-context branch January 5, 2018 15:52
yenshih pushed a commit to yenshih/react that referenced this pull request Jan 6, 2018
* pass host context to finalizeInitialChildren

* don't retrieve context an extra time
ManasJayanth pushed a commit to ManasJayanth/react that referenced this pull request Jan 12, 2018
* pass host context to finalizeInitialChildren

* don't retrieve context an extra time
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.

4 participants