Skip to content

Sema: Error for potentially unavailable conformances in Swift 6 #70730

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

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented Jan 5, 2024

Diagnostics about use of potentially unavailable conformances had to be downgraded to warnings in Swift 5 in order to preserve source compatibility. These diagnostics should be errors by default in Swift 6.

Now that the diagnostics are automatically errors in Swift 6, we don't need an -enable-conformance-availability-errors flag to control whether unavailable conformances are diagnosed as errors. Nobody was using the flag so it should be safe to remove.

Resolves rdar://88210812

Diagnostics about use of potentially unavailable conformances had to be
downgraded to warnings in Swift 5 in order to preserve source compatibility.
These diagnostics should be errors by default in Swift 6.

Resolves rdar://88210812
Now that the diagnostics are automatically errors in Swift 6, we don't need an
`-enable-conformance-availability-errors` flag to control whether unavailable
conformances are diagnosed as errors. Nobody was using the flag so it should be
safe to remove.

Part of rdar://88210812
@tshortli tshortli force-pushed the potentially-unavailable-conformance-error-swift6 branch from 467f767 to 41c4143 Compare January 5, 2024 01:53
@tshortli
Copy link
Contributor Author

tshortli commented Jan 5, 2024

@swift-ci please test

@tshortli tshortli requested review from xymus and beccadax January 5, 2024 01:54
@tshortli tshortli merged commit 556a9b1 into swiftlang:main Jan 5, 2024
@tshortli tshortli deleted the potentially-unavailable-conformance-error-swift6 branch January 5, 2024 16:47
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.

2 participants