Skip to content

Conversation

@bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Apr 24, 2017

This will enable a React-to-ReactNative sync that is currently blocked by a stack-compatible-only test renderer forked from PR #7516 that's used in fbsource. Once the changes proposed in #7516 have been made to test renderer this target can be removed from the bundle (and the forked renderer can be deleted as well).

This PR also fixes a typo in the temporary checkPropTypes accessor that that had been added to the React object previously.

Also fixed an error in a temporary export property that had been added to the React object
Copy link
Collaborator

@acdlite acdlite left a comment

Choose a reason for hiding this comment

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

I trust you

@bvaughn bvaughn merged commit e71b308 into facebook:master Apr 25, 2017
@bvaughn bvaughn deleted the test-renderer-stack branch April 25, 2017 01:45
@bvaughn
Copy link
Contributor Author

bvaughn commented Apr 25, 2017

I trust you

Jafar laughing

Muwahahaha!

);
warnedForCheckPropTypes = true;
return ReactPropTypes;
return checkPropTypes;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did this typo get into any non-alpha release?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope! I fat fingered it in PR #9399 in prep for alpha 10.

NMinhNguyen referenced this pull request in enzymejs/react-shallow-renderer Jan 29, 2020
Also fixed an error in a temporary export property that had been added to the React object
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