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 react-tools #4010

Merged
merged 2 commits into from
Jul 30, 2015
Merged

Remove react-tools #4010

merged 2 commits into from
Jul 30, 2015

Conversation

zpao
Copy link
Member

@zpao zpao commented Jun 2, 2015

ReactTools is deprecated so lets remove it.

@zpao
Copy link
Member Author

zpao commented Jun 15, 2015

Updated with the complete removal of react-tools. I left a couple of the other changes in-tact and then re-added jstransform and commoner to our devdeps (I hadn't done a fresh npm install to test last time…). We'll get rid of jstransform once we do #4130.

@zpao zpao changed the title Move react-tools from root. Remove react-tools Jun 15, 2015
@zpao
Copy link
Member Author

zpao commented Jun 15, 2015

cc @amasad. If react-native isn't ready to consume react from npm, we could build a new package that is just src/ (maybe call it react-provides-module or something similarly obtuse). I know there are some other pieces that we'll need for react-native before it's ready - @sebmarkbage is working on that.

@zpao zpao force-pushed the really-deprecate-react-tools branch from 91a095c to bff47b9 Compare June 15, 2015 21:08
@amasad
Copy link
Contributor

amasad commented Jun 15, 2015

We can't because of the overrides?

@zpao
Copy link
Member Author

zpao commented Jun 15, 2015

Yea, pretty much. But decoupling the overrides is not completely straightforward.

zpao added 2 commits July 30, 2015 10:15
ReactTools is deprecated, it doesn't belong at the root of the project. We'll remove it after 0.14 but for now this moves it so that it's contained and not intermixed with the rest of the project. The currect behavior of copying src/ into the package is maintained.
@zpao zpao force-pushed the really-deprecate-react-tools branch from d6bdd99 to 94bc29b Compare July 30, 2015 17:15
zpao added a commit that referenced this pull request Jul 30, 2015
@zpao zpao merged commit 3d197e2 into facebook:master Jul 30, 2015
@zpao zpao deleted the really-deprecate-react-tools branch October 13, 2015 16:41
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