-
-
Notifications
You must be signed in to change notification settings - Fork 237
Fix react and react-native deps #124
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
Conversation
This package don't need |
On the other hand we're using gitflow now, so please create PR to |
I agree there is no direct dependency on |
01c4476
to
1010c4b
Compare
I've changed the base branch and rebased onto |
I mean, what script of the package requires that dependency? We're using the example project to debug the plugin and we don't need that :) |
I can remove it if you really want to.. It will cause a warning when installing dependencies:
I have a bunch of other branches locally, including one related to CI which builds the library code in isolation (no other code required), to be able to detect and fix issues. Having a complete description of the project dependency model be one of the prerequisites. |
Hi @jdnichollsc - Are you no longer interested in the fixes here? |
@friederbluemle if you want, please fix the merge issues of these devDependencies and I can merge that |
@jdnichollsc Sure thing! Can you please reopen the PR, and I'll push the updated branch with conflicts resolved. |
1010c4b
to
03bd173
Compare
Thanks! Updated 👍 |
03bd173
to
9df5d27
Compare
Thanks to you! |
Fix react and react-native deps
Use fixed versions for
react
andreact-native
.One version of
react-native
is always tightly coupled with exactly one version ofreact
. They can't go out of sync, or things will likely break.