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

update react-tools to ~0.4.0 #3

Merged
merged 1 commit into from Jul 31, 2013
Merged

update react-tools to ~0.4.0 #3

merged 1 commit into from Jul 31, 2013

Conversation

hmil
Copy link
Contributor

@hmil hmil commented Jul 30, 2013

This uses the latest react-tools version.
Currently, using grunt-react with react lib v0.4.0 in the browser prints warning messages and prevents internal optimizations.
Tests passes, this is not a major version, it shouldn't break...

@ericclemmons
Copy link
Owner

React just updated to 0.4.1. Should we use that version instead?

@hmil
Copy link
Contributor Author

hmil commented Jul 31, 2013

It's tilde version range so if you try now, you get 0.4.1 and this should work until next minor (see https://npmjs.org/doc/json.html#Tilde-Version-Ranges ). You could use 0.x since minor are not supposed to break either but it's a bit risky. Moreover, this allows you to bump grunt-react version number accordingly so we know that using grunt-react 0.4.x uses react-tools 0.4.x

@ericclemmons
Copy link
Owner

Oh, that's right! I use them myself, but I still find myself of being in the habit of explicitly setting the version :)

Anyway, thanks for the PR. I'm merging down now!

ericclemmons added a commit that referenced this pull request Jul 31, 2013
update react-tools to ~0.4.0
@ericclemmons ericclemmons merged commit 4a5f4a5 into ericclemmons:master Jul 31, 2013
@ericclemmons
Copy link
Owner

Now at grunt-react@v0.4.0, just because I like both projects to be on 0.4.x :D

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.

None yet

2 participants