Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily add stack target to react-test-renderer bundle #9514

Merged
merged 1 commit into from
Apr 25, 2017

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!

@@ -113,7 +113,7 @@ if (__DEV__) {
'(https://fb.me/migrating-from-react-proptypes)',
);
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