From 69512d871dd190f549ae1adf0b7659a2d184c8ec Mon Sep 17 00:00:00 2001 From: Lawrence Forooghian Date: Thu, 1 Aug 2024 11:57:00 +0100 Subject: [PATCH] try running on platforms too --- .github/workflows/check.yaml | 11 +++++++++++ CONTRIBUTING.md | 2 ++ 2 files changed, 13 insertions(+) diff --git a/.github/workflows/check.yaml b/.github/workflows/check.yaml index 269ae3e..e4d4833 100644 --- a/.github/workflows/check.yaml +++ b/.github/workflows/check.yaml @@ -18,3 +18,14 @@ jobs: # https://forums.swift.org/t/warnings-as-errors-for-libraries-frameworks/58393/2 - run: swift build -Xswiftc -warnings-as-errors - run: swift test + + check-on-platforms: + runs-on: macos-latest + strategy: + matrix: + platform: [macOS, iOS, tvOS] + steps: + # TODO OS version, and ambiguity in platforms, and no match + - run: xcodebuild -list + - run: xcodebuild -scheme AblyChat -destination "platform=${{ matrix.platform }}" + - run: xcodebuild test -scheme AblyChat -destination "platform=${{ matrix.platform }}" diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 2f05361..e985a74 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -29,3 +29,5 @@ At time of writing, the latest version of Xcode is 15.4, and the `Package.swift` and then we also take the minimum of these and ably-cocoa (which at time of writing were all lower than the above) TODO let's see what Xcode 16 says too though — OK, doesn’t seem any different in Beta 3 + +TODO we probably also have to take into account which simulators are available on the version of Xcode we're using, _and_ which are installed on the GitHub runner