Skip to content

Fix peer dependencies warnings#2480

Merged
alvelig merged 1 commit intoreact-native-maps:masterfrom
junkpiano:refactor/react-native-56
Sep 13, 2018
Merged

Fix peer dependencies warnings#2480
alvelig merged 1 commit intoreact-native-maps:masterfrom
junkpiano:refactor/react-native-56

Conversation

@junkpiano
Copy link
Copy Markdown
Contributor

@junkpiano junkpiano commented Sep 10, 2018

Does any other open PR do the same thing?

No.

What issue is this PR fixing?

This PR will fix peer dependencies warnings when you run npm install on the projects which are using React Native 0.56.0(or later). This relates to #2332 .

How did you test this PR?

in my project,

  • remove react-native-maps from package.json.
  • remove node_modules
  • run npm install again
  • run npm install <relative path to react-native-maps> --save
  • confirm warning messages related to react-native don't appear.

However, it doesn't solve the warning around @babel-core@7.0.0-beta.38(required by @babel/plugin-check-constants@7.0.0-beta.38).

- update peer dependencies
- modified version notation of react-native.
- use the release version of react
@junkpiano junkpiano force-pushed the refactor/react-native-56 branch from 8b3183c to 47867bb Compare September 10, 2018 11:40
@rborn
Copy link
Copy Markdown
Collaborator

rborn commented Sep 10, 2018

LGTM @alvelig 🐽

@rborn
Copy link
Copy Markdown
Collaborator

rborn commented Sep 13, 2018

@alvelig ping ? 🤗

@alvelig alvelig merged commit d21749a into react-native-maps:master Sep 13, 2018
pinpong pushed a commit to pinpong/react-native-maps that referenced this pull request Feb 28, 2025
…ct-native-56

Fix peer dependencies warnings
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.

3 participants