Skip to content

Conversation

@rnro
Copy link
Contributor

@rnro rnro commented May 8, 2025

Motivation:

Building for visionOS no longer seems to be having issues on the runners, we should enable it to increase our test coverage.

###Modifications:

Enable visionOS xcodebuilds by default in CI

Result:

NIO and downstream repositories will see their existing macOS CI workflows now build for visionOS.

They can be disabled if necessary by passing visionos_xcode_build_enabled: false

Motivation:

Building for visionOS no longer seems to be having issues on the
runners, we should enable it to increase our test coverage.

Modifications:

Enable visionOS xcodebuilds by default in CI

Result:

NIO and downstream repositories will see their *existing* macOS CI
workflows now build for visionOS.
@rnro rnro added the semver/none No version bump required. label May 8, 2025
@rnro rnro enabled auto-merge (squash) May 8, 2025 08:01
@rnro rnro merged commit 00fedd6 into apple:main May 8, 2025
41 of 42 checks passed
rnro added a commit that referenced this pull request May 9, 2025
glbrntt pushed a commit that referenced this pull request May 9, 2025
Reverts #3234

This seems to have caused some instability which I'll investigate so we
should revert for now.
rnro added a commit that referenced this pull request May 13, 2025
### Motivation:

Building for visionOS no longer seems to be having issues on the
runners, we should enable it to increase our test coverage.

### Modifications:

Enable visionOS xcodebuilds by default in CI

### Result:

NIO and downstream repositories will see their *existing* macOS CI
workflows now build for visionOS.

They can be disabled if necessary by passing
`visionos_xcode_build_enabled: false`


See #3234, #3235
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.

2 participants