Skip to content

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

Merged
merged 1 commit into from
Feb 20, 2020

Conversation

friederbluemle
Copy link
Contributor

Use fixed versions for react and react-native.

One version of react-native is always tightly coupled with exactly one version of react. They can't go out of sync, or things will likely break.

@friederbluemle friederbluemle mentioned this pull request Nov 16, 2019
@jdnichollsc
Copy link
Member

This package don't need react as dependency, and for that reason don't care the version used for React Native 🙂

@jdnichollsc
Copy link
Member

On the other hand we're using gitflow now, so please create PR to develop branch 👍

@friederbluemle
Copy link
Contributor Author

I agree there is no direct dependency on react in the source code. But there is one (of course) on react-native. And react-native has a peer dependency on react. Without declaring it as a dev dependency, it would be missing sources when resolving types located in the react package (during development).

@friederbluemle friederbluemle changed the base branch from master to develop November 16, 2019 03:55
@friederbluemle
Copy link
Contributor Author

I've changed the base branch and rebased onto develop. 👍

@jdnichollsc
Copy link
Member

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 :)

@friederbluemle
Copy link
Contributor Author

I can remove it if you really want to.. It will cause a warning when installing dependencies:

warning " > react-native@0.60.0" has unmet peer dependency "react@16.8.6".

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.

@friederbluemle
Copy link
Contributor Author

Hi @jdnichollsc - Are you no longer interested in the fixes here?

@jdnichollsc
Copy link
Member

@friederbluemle if you want, please fix the merge issues of these devDependencies and I can merge that

@friederbluemle
Copy link
Contributor Author

friederbluemle commented Feb 20, 2020

@jdnichollsc Sure thing! Can you please reopen the PR, and I'll push the updated branch with conflicts resolved.

@friederbluemle
Copy link
Contributor Author

Thanks! Updated 👍

@jdnichollsc jdnichollsc merged commit bb2b1fc into proyecto26:develop Feb 20, 2020
@jdnichollsc
Copy link
Member

Thanks to you!

@friederbluemle friederbluemle deleted the fix-deps branch February 21, 2020 00:24
phuongwd pushed a commit to phuongwd/react-native-inappbrowser that referenced this pull request Jul 31, 2020
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.

2 participants