Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Mark retroactive conformances appropriately. #2569

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

Lukasa
Copy link
Contributor

@Lukasa Lukasa commented Oct 23, 2023

Motivation:

In nightly Swift, we now need to mark retroactive conformances when they are intentional. These conformances are safe for us, so we can safely suppress the warnings.

Modifications:

  • Mark NIOFoundationCompat retroactive conformances.

Result:

Nightly builds work again

@Lukasa Lukasa added the 🔨 semver/patch No public API change. label Oct 23, 2023
Motivation:

In nightly Swift, we now need to mark retroactive conformances when
they are intentional. These conformances are safe for us, so we can
safely suppress the warnings.

Modifications:

- Mark NIOFoundationCompat retroactive conformances.

Result:

Nightly builds work again
@Lukasa Lukasa force-pushed the cb-retroactive-conformances branch from 0ecac73 to 008fc29 Compare October 23, 2023 15:49
@weissi weissi self-requested a review October 23, 2023 15:56
Copy link
Member

@weissi weissi left a comment

Choose a reason for hiding this comment

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

thank you!

@Lukasa Lukasa merged commit 86d05fb into apple:main Oct 23, 2023
@Lukasa Lukasa deleted the cb-retroactive-conformances branch October 23, 2023 18:00
adam-fowler pushed a commit to adam-fowler/swift-nio that referenced this pull request Oct 24, 2023
Motivation:

In nightly Swift, we now need to mark retroactive conformances when
they are intentional. These conformances are safe for us, so we can
safely suppress the warnings.

Modifications:

- Mark NIOFoundationCompat retroactive conformances.

Result:

Nightly builds work again
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants