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

Remove createClass, PropTypes, DOM factories, and createMixin from React object #9788

Merged
merged 1 commit into from
May 30, 2017

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented May 26, 2017

There's still a few callers in www which I'll fix internally before merging.

We also have some RN code that still uses them but plan to either codemod it or shim it for the time being. In either case this will only matter after we cut another react package and update internal RN code to use it, which probably won't happen before codemodding.

@gaearon gaearon force-pushed the remove-cc branch 2 times, most recently from d5188d2 to a26fcbf Compare May 26, 2017 14:47
@flarnie flarnie self-requested a review May 27, 2017 16:58
@flarnie
Copy link
Contributor

flarnie commented May 27, 2017

I was just making an issue to track this, and then saw that you were already working on it. Nice. :)

Copy link
Contributor

@flarnie flarnie left a comment

Choose a reason for hiding this comment

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

I love the sound of code being deleted~

},
"react.production.min.js (UMD_PROD)": {
"size": 15753,
"gzip": 5824
"size": 7756,
Copy link
Contributor

Choose a reason for hiding this comment

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

⚡️ 🎸 🥇

Copy link
Contributor

@nhunzaker nhunzaker left a comment

Choose a reason for hiding this comment

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

Not really sure I can approve this, but it makes me happy!

@gaearon
Copy link
Collaborator Author

gaearon commented May 30, 2017

Internal callers in www should be patched up, and we'll need to codemod fbsource soon anyway.
Seems fine to merge.

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.

None yet

4 participants