Skip to content

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