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

react-core npm module #232

Closed
wants to merge 1 commit into from
Closed

react-core npm module #232

wants to merge 1 commit into from

Conversation

zpao
Copy link
Member

@zpao zpao commented Jul 24, 2013

Jordan wanted an easy way to use React, and specifically behave nicely
with browserify. He has been using
https://github.com/jordwalke/npm-react-core as a placeholder, but in
order to keep doing that, it requires another build step and more places
for us to screw up. This just re-uses the NPM module we're already
shipping and is really easy to update.

Note: we may want to pull this out into it's own project eventually, but
for now... move fast?

Jordan wanted an easy way to use React, and specifically behave nicely
with browserify. He has been using
https://github.com/jordwalke/npm-react-core as a placeholder, but in
order to keep doing that, it requires another build step and more places
for us to screw up. This just re-uses the NPM module we're already
shipping and is really easy to update.

Note: we may want to pull this out into it's own project eventually, but
for now... move fast?
@zpao
Copy link
Member Author

zpao commented Jul 24, 2013

@jordwalke I made https://github.com/facebook/react/wiki/Release-Process when we launched, though it needs a little bit of tweaking (and would need more if we do this)

@zpao zpao mentioned this pull request Oct 23, 2013
@zpao
Copy link
Member Author

zpao commented Nov 14, 2013

Going to close this out. I liked the other route better anyway.

@zpao zpao closed this Nov 14, 2013
@zpao zpao deleted the react-core branch January 14, 2014 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants