-
Notifications
You must be signed in to change notification settings - Fork 697
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
Update headers for RN@0.40.0 #487
Conversation
React Native has made a breaking change to header paths in [0.40.0](https://github.com/facebook/react-native/releases/tag/v0.40.0). Besides fixing the headers, this adds `RN@0.40` and `React@15.4.0` as peer depedencies and bump version.
Is adding react and react native as a peer dependency necessary? |
It breaks compatibility with older version of React Native. For this particular PR, React Native 0.40 is required, so I added it as peer dependency. |
@sntran I'm still having compilation error about <React/RCTConvert+CoreLocation.h> file not found (within RCTMapboxGLConversions.h> with this PR, any other path/settings i should alter? |
@sntran Maybe it is because CocoaPods settings were not changed? |
I have always manually installed for iOS so I don't know about CocoaPods. I'll see what I can do. |
@sntran Thanks - right, i've just checked with manual install - it works well! Unfortunately some libraries like latest iOS XMPP Framework have only CocoaPods install option... |
Looks like RN0.40 cannot be installed via CocoaPods at all now - 'yoga' library is missed (PR is in progress as i see). |
Thanks! I think the cocoapod failures could be fixed with #499 |
Any updates when can we have a new release? |
React Native has made a breaking change to header paths in 0.40.0.
Besides fixing the headers, this adds
RN@0.40
andReact@15.4.0
aspeer dependencies and bump version.
This PR simply adjusts the headers.