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

Bump iOS Deployment Target to 9 & Fix Deprecations #82

Closed

Conversation

JustinDSN
Copy link
Contributor

@JustinDSN JustinDSN commented Aug 19, 2020

Because Xcode 12 drops support for iOS 8 deployment target, we need to upgrade Aardvark's deployment target.

  • Bumps iOS Deployment Target to 9
  • Migrate from deprecated APIs
  • Updates to Swift 5
  • Update xcodeproj to recommended settings

@JustinDSN JustinDSN requested a review from dfed August 19, 2020 17:27
@dfed
Copy link
Collaborator

dfed commented Aug 19, 2020

Hi @JustinDSN! Thanks for your interest in Aardvark 🙂

The changes proposed here would require a major version bump, since we'd be dropping functionality. Can you file an Issue for Xcode 12 support? We'll likely want to roll these changes in with other breaking changes.

@JustinDSN
Copy link
Contributor Author

Resolves #83

README.md Outdated Show resolved Hide resolved
@NickEntin NickEntin self-requested a review August 19, 2020 19:18
Aardvark.podspec Outdated Show resolved Hide resolved
@NickEntin
Copy link
Collaborator

@JustinDSN If you rebase on master, you should be able to get a green build now using GitHub Actions. There will be some merge conflicts (the change in CI setup and updates to the sample app), but the flaky test has been disabled, so you should see stable builds.

@NickEntin
Copy link
Collaborator

This PR is obsoleted by #92

@NickEntin NickEntin closed this Dec 9, 2020
@JustinDSN
Copy link
Contributor Author

Thanks for getting to this @NickEntin fell too far on my todo list.

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.

None yet

3 participants