-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix react-router dependency. Fix #439. #441
Conversation
@@ -19,7 +19,7 @@ | |||
"react": "^0.14.0-rc", | |||
"react-dom": "^0.14.0-rc", | |||
"react-relay": "file:../../", | |||
"react-router": "^1.0.0-rc1", |
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.
I think this should install rc3 as is (at least with a fresh install or npm update
)
Edit: sorry, misread the other issue and I see that we actually need rc3 so it's a safe move to specify that.
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.
@zpao Do you mean make it precisely 1.0.0-rc3
?
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.
No, I think this is fine, I just misunderstood the reason for the change.
@facebook-github-bot import |
Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/1028373460535842/int_phab to review. |
c675918
@octref: Yes, saw that, but was easy to resolve. |
No description provided.