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

make react-tools a peer dependency #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

nelix
Copy link

@nelix nelix commented Jan 8, 2015

this will save some time bumping versions every time a new version of react is released, fixes #29

this will save some time bumping versions every time a new version of react is released, fixes petehunt#29
@waldreiter
Copy link

Yes! This is one of the reasons why I use a homemade loader for JSX.

The version < 2 seems random to me. I just use "*".

@nelix
Copy link
Author

nelix commented Jan 8, 2015

I'm not really an npm pro, but it needs to be over 0.12.1 but under version 1? But for some reason I typed 2.
So I changed it to 1.

@nelix
Copy link
Author

nelix commented Feb 12, 2015

:(

@benmosher
Copy link

This, please. 👍

Otherwise, it is difficult to ensure the desired version of react-tools is used for the JSX load, as npm will install the most current version, which may be out of sync with the project's React version.

@coodoo
Copy link

coodoo commented Apr 9, 2015

+1

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.

Move react-tools to peerDependencies?
5 participants