Skip to content

Explicitly add visionOS availability to tests that check availability. #424

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

Merged
merged 1 commit into from
May 21, 2024

Conversation

briancroom
Copy link
Contributor

Since swiftlang/swift#72834, the Swift compiler supports visionOS availability annotations. For completeness, add this platform to existing tests which declare Apple platform availability.

@briancroom
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@grynspan grynspan left a comment

Choose a reason for hiding this comment

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

:shipit:

@grynspan
Copy link
Contributor

Merging on Brian's behalf.

@grynspan grynspan merged commit 406564e into main May 21, 2024
2 of 3 checks passed
@grynspan grynspan deleted the bcroom/visionOS-availability branch May 21, 2024 17:16
grynspan added a commit that referenced this pull request May 21, 2024
grynspan added a commit that referenced this pull request May 21, 2024
…lability. (#424)" (#426)

This reverts commit 406564e.

The revert is needed because Swift 5.10 does not recognize visionOS and
swift-testing does not currently have the ability to conditionalize
availability with `#if compiler(>=5.11)`.

We can reapply the change in the future if/when we require Swift 6.

### Checklist:

- [x] Code and documentation should follow the style of the [Style
Guide](https://github.com/apple/swift-testing/blob/main/Documentation/StyleGuide.md).
- [x] If public symbols are renamed or modified, DocC references should
be updated.
briancroom added a commit that referenced this pull request Jun 13, 2024
… (Take 2) (#477)

Re-attempt the change from
#424 now that we require a
Swift 6 toolchain to building the package.
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.

3 participants