Skip to content

Conversation

@rnro
Copy link
Contributor

@rnro rnro commented Mar 19, 2025

Motivation:

We are still exploring the shape of macOS CI and exploring the system's capabilities.

Modifications:

  • Add visionOS jobs
  • Remove Xcode 15.4 as it is no longer on the runners
  • Add support for running tests targetting other platforms
  • Enable test targeting other platforms on daily jobs to increase coverage without bloating PR runtimes.

Result:

More macOS CI coverage

CI run with this enabled https://github.com/apple/swift-nio/actions/runs/13944231147/job/39027647470?pr=3150 - the iOS failure is unrelated to this change

@rnro rnro added the semver/none No version bump required. label Mar 19, 2025
@rnro rnro force-pushed the update_macos_ci branch 3 times, most recently from 14b4582 to 9279175 Compare March 19, 2025 14:32
@rnro rnro marked this pull request as ready for review March 19, 2025 14:33
Motivation:

We are still exploring the shape of macOS CI and exploring the system's
capabilities.

Modifications:

* Add visionOS jobs
* Remove Xcode 15.4 as it is no longer on the runners
* Add support for running tests targetting other platforms
* Enable test targetting other platforms on daily jobs to increase
  coverage without bloating PR runtimes.

Result:

More macOS CI coverage
@rnro rnro force-pushed the update_macos_ci branch from 9279175 to b699831 Compare March 19, 2025 14:33

on:
workflow_call:
inputs:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably for a separate PR, but should we rename from macos_tests to, for example xcode_tests or apple_tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I'm not sure - I hovered on darwin_tests too. I'll ask around.

@rnro rnro merged commit 6ff70b5 into main Mar 20, 2025
38 of 42 checks passed
@rnro rnro deleted the update_macos_ci branch March 20, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants