Skip to content

Remove var#11752

Closed
raphamorim wants to merge 10 commits intofacebook:masterfrom
raphamorim:remove-var
Closed

Remove var#11752
raphamorim wants to merge 10 commits intofacebook:masterfrom
raphamorim:remove-var

Conversation

@raphamorim
Copy link
Copy Markdown
Contributor

Ref: #11736

checked={true}
onChange={() => {}}
/>
<input type="checkbox" ref={n => (this.b = n)} checked={true} />
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is undoing a recently merged change.

Copy link
Copy Markdown
Contributor Author

@raphamorim raphamorim Dec 3, 2017

Choose a reason for hiding this comment

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

damn, you're right. I'll solve it, can you continue review the other files?
I had conflict only with this file and the ReactFiber of reconciler package

@raphamorim raphamorim force-pushed the remove-var branch 3 times, most recently from 5cc2d2a to d61ed65 Compare December 3, 2017 00:37
@raphamorim
Copy link
Copy Markdown
Contributor Author

raphamorim commented Dec 3, 2017

Hm, I discovery other inconsistencies @gaearon (my mistake 😞 )
I'll solve it and update here.

@raphamorim raphamorim force-pushed the remove-var branch 3 times, most recently from e7d5217 to 51c4f3e Compare December 3, 2017 02:11
@raphamorim
Copy link
Copy Markdown
Contributor Author

Fixed @gaearon

@gaearon
Copy link
Copy Markdown
Collaborator

gaearon commented Dec 4, 2017

Sorry, I merged a few things and it's out of date again. Maybe we can split this further?

@raphamorim
Copy link
Copy Markdown
Contributor Author

Sure, I break in other Pull Requests.

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.

3 participants