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

build for latest iOS/watchOS #642

Closed
wants to merge 16 commits into from
Closed

Conversation

cbaker6
Copy link
Contributor

@cbaker6 cbaker6 commented Feb 17, 2022

Separated out this PR for easier review.

  • Bumps CI to Xcode 14, iOS 16, watchOS 9.0
  • Fix Xcode 14 warnings about switching open->public
  • Fix library evolutionary support warnings in workspace build
  • Enable async/await methods on iOS13+ and watchOS6+
  • Minimum deployment of watchOS is 6.0 since there's no dependencies on a higher OS (Some numbers were out of alignment here so I put everything on 6.0)
  • Bump minimum tools in Package.swift to 5.5 since the async/await methods requires this as a minimum. This also always for DocC documentation to be added. I'm assuming a lot of the README can be copied to DocC in the future, but I didn't add any of these files
  • Removed periods at the end of bullet points in README
  • Add watchOS builds to CI to protect against breaking PR's. These should be able to run tests in Xcode 13 and 14, but the Xcode project isn't configured for this
  • allow OCKCalendarController and OCKWeekCalendarController view models to build on watchOS. These were using import UIKit, but can use import CoreGraphics instead to build
  • Add .spi.yml for properly building on Swift Package Index. Swift Package Index is the successor of SwiftPM Library. Also added badges:
    image

Note1: The watchOS platform should appear after this PR is merged to the main as the added .spi.yml tells Swift Package Index what scheme to use for building watch.

@cbaker6
Copy link
Contributor Author

cbaker6 commented Mar 5, 2022

@gavirawson-apple when you get a chance, take a look and let me know if any more changes are needed

@cbaker6 cbaker6 closed this Oct 9, 2022
@cbaker6 cbaker6 reopened this Oct 9, 2022
@cbaker6
Copy link
Contributor Author

cbaker6 commented Oct 21, 2022

@gavirawson-apple I updated the list of changes to the repo

@cbaker6 cbaker6 closed this Apr 7, 2023
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.

1 participant