-
Notifications
You must be signed in to change notification settings - Fork 69
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
Upgrade to react 16 #57
Conversation
Signed-off-by: Stephane Jolicoeur <sjolicoeur@pivotal.io>
Signed-off-by: Reid Mitchell <rmitchell@pivotal.io>
package.json
Outdated
@@ -34,8 +34,8 @@ | |||
"tether": "^1.3.7" | |||
}, | |||
"peerDependencies": { | |||
"react": "^0.14.0 || ^15.0.0", | |||
"react-dom": "^0.14.0 || ^15.0.0" | |||
"react": "^16.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Support for 0.14 and 15.x probably shouldn't be removed unless there is a breaking change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rafeememon ok will fix this ASAP.
…aking change Signed-off-by: Reid Mitchell <rmitchell@pivotal.io>
LGTM! |
Is there anything blocking this PR? |
@rafeememon we made the necessary change, is it possible to have the PR merged and a new release? |
Any update on this? |
It's not going to happen until #58 is fixed |
@souporserious are we really blocked on merging this because there isn't a new maintainer assigned yet? I just want to verify |
We've made changes to the package.json to allow react-tether to work with React v16.0.0
this should fix #56
PS: we aren't sure if the yarn.lock is necessary for the upgrade.