Skip to content
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

chore: update ios deployment target min to 9 (react-native min) #112

Merged
merged 1 commit into from May 31, 2019

Conversation

mikehardy
Copy link
Contributor

Overview

This is a trivial patch - but react-native has an iOS minimum target of 9, and with this module set to 8 it generates a warning during the build

Insert ramble about broken windows theory with an add-on related to log spam here...

Note that all of this textual change was the result of simply opening the XCode project, going to the info area, and altering the minimum from 8 to 9 - it spiders out into a couple places at once and tries some i18n thing πŸ€·β€β™‚οΈ

Test Plan

@mikehardy
Copy link
Contributor Author

Also note that we were just bitten (on react-native-share) with a build fail that was release-target only. And, I was also bitten recently on a build failure for tvOS only. Ugh. I mention this because you might want to add the matrix of build targets for your example to CircleCI - not just 1, but the full 2x2 of debug or release x iOS or tvOS.

@matt-oakes matt-oakes merged commit c05df9b into react-native-netinfo:master May 31, 2019
@matt-oakes
Copy link
Collaborator

Thanks for the heads up about the build matrix. I'll take a look into that.

@mikehardy mikehardy deleted the ios-deploy-min-9 branch May 31, 2019 16:43
@mikehardy
Copy link
Contributor Author

when you init an example app they give you a hint on how to run the react-native CLI targeting other schemes, usually it's just adding --scheme example-tvOS if I recall correctly. And I think you can set a release target too but I haven't done that yet (which is why were bitten by it :-) )

react-native-community-bot pushed a commit that referenced this pull request Jun 1, 2019
## [3.2.1](v3.2.0...v3.2.1) (2019-06-01)

### Bug Fixes

* Update iOS deployment target min to avoid a warning ([#112](#112) by [@mikehardy](https://github.com/mikehardy)) ([c05df9b](c05df9b))
@react-native-community-bot
Copy link
Collaborator

πŸŽ‰ This PR is included in version 3.2.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants