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

Update iOS deployment target to silence Xcode 15 warnings #1589

Closed
wants to merge 4 commits into from

Conversation

ksuther
Copy link
Contributor

@ksuther ksuther commented Sep 19, 2023

Changes in this pull request

Xcode 15 has moved the iOS deployment target up to iOS 12 (just like #1573 for Xcode 14). I did not modify the changelog because there's already an entry there for iOS 11 and macOS 10.13.

This also includes a build error fix for IGWarnAssert (dffcd4d). I tried running the unit tests but a number of them are failing from previous changes.

Checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
  • I have reviewed the contributing guide

Xcode 15 doesn't support deploying to older versions and shows a warning on every build
- BuildFile header reference no longer exists
- IGListSingleSelectionControllerTests.m was in compile sources twice
@facebook-github-bot
Copy link
Contributor

@TimOliver has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@TimOliver merged this pull request in 7256bfb.

GreeMoz pushed a commit to appintheair/IGListKit that referenced this pull request Jan 31, 2024
…1589)

Summary:
## Changes in this pull request

Xcode 15 has moved the iOS deployment target up to iOS 12 (just like Instagram#1573 for Xcode 14). I did not modify the changelog because there's already an entry there for iOS 11 and macOS 10.13.

This also includes a build error fix for IGWarnAssert (dffcd4d). I tried running the unit tests but a number of them are failing from previous changes.

### Checklist

- [ ] All tests pass. Demo project builds and runs.
- [x] I added tests, an experiment, or detailed why my change isn't tested.
- [ ] I added an entry to the `CHANGELOG.md` for any breaking changes, enhancements, or bug fixes.
- [x] I have reviewed the [contributing guide](https://github.com/Instagram/IGListKit/blob/main/.github/CONTRIBUTING.md)

Pull Request resolved: Instagram#1589

Reviewed By: fabiomassimo

Differential Revision: D50013207

Pulled By: TimOliver

fbshipit-source-id: 6c7a7b2ca3dd1265f11e85979432eded3292ed06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants