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

Support react v15 #8

Merged
merged 1 commit into from Mar 18, 2016
Merged

Support react v15 #8

merged 1 commit into from Mar 18, 2016

Conversation

lo1tuma
Copy link
Contributor

@lo1tuma lo1tuma commented Mar 18, 2016

No description provided.

@lxanders
Copy link
Collaborator

I'm not sure about this. We don't win anything right now of using a release candidate, do we? Then again, I'm not against it so if you consider this still useful, I will merge.

@lo1tuma
Copy link
Contributor Author

lo1tuma commented Mar 18, 2016

We do not use a specific version of react anyway, because we defined it as peer dependency. This PR just increases the range so users of react-google-tag-manger can use it with react v15.

Of course we use a specific react version in our tests, but I don’t see any problem with using the release candidate version there.

@lxanders
Copy link
Collaborator

You are absolutely right, this makes sense. Better be prepared for people upgrading react 👍.

lxanders pushed a commit that referenced this pull request Mar 18, 2016
@lxanders lxanders merged commit b7746b0 into master Mar 18, 2016
@lxanders lxanders deleted the react-15 branch March 18, 2016 19:39
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