-
Notifications
You must be signed in to change notification settings - Fork 40
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
Bump iOS bridge versions to 0.5.4 #207
Conversation
a97640e
to
27777c8
Compare
@@ -2,7 +2,6 @@ require_relative '../node_modules/react-native/scripts/react_native_pods' | |||
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules' | |||
|
|||
platform :ios, '11.0' | |||
use_frameworks! |
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.
use_frameworks!
is not supported by the latest version of the bridge
@@ -3784,7 +3784,16 @@ prompts@^2.4.0: | |||
kleur "^3.0.3" | |||
sisteransi "^1.0.5" | |||
|
|||
prop-types@15.x.x, prop-types@^15.7.2: | |||
prop-types@15.x.x: |
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 need to understand why some changes happened here
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.
probably something missed in a previous PR
@@ -202,90 +202,22 @@ | |||
shellPath = /bin/sh; | |||
shellScript = "export NODE_BINARY=node\n../node_modules/react-native/scripts/react-native-xcode.sh"; | |||
}; | |||
0F5DD2465E82C89D1BBC8144 /* [CP] Embed Pods Frameworks */ = { | |||
1C1ED57B55ECBF1430A95142 /* [CP] Copy Pods Resources */ = { |
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 need to understand why some changes happened here
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.
This is because we removed !use_frameworks
:)
What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)