Skip to content
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

Swift: Fix for OptionSet and BinaryInteger models #18154

Merged
merged 3 commits into from
Nov 29, 2024

Conversation

geoffw0
Copy link
Contributor

@geoffw0 geoffw0 commented Nov 28, 2024

Small fix for the OptionSet and BinaryInteger models for Swift 6. I'm particularly displeased with this one as there doesn't seem to be much reason why BinaryInteger would apparently not derive from CustomStringConvertible any more (hence needing its own model of .description), though it does remind me of some of the changes in #17989 to integral type models. Similarly for OptionSet, either it's been divorced from RawRepresentable or for some reason we're not extracting the association correctly any more.

In any case, all of the test regressions I'm aware of are now fixed.

@geoffw0 geoffw0 added no-change-note-required This PR does not need a change note Swift labels Nov 28, 2024
@geoffw0 geoffw0 requested a review from a team as a code owner November 28, 2024 18:54
Copy link
Contributor

@redsun82 redsun82 left a comment

Choose a reason for hiding this comment

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

Happy to reapprove after fixing the formatting!

@geoffw0
Copy link
Contributor Author

geoffw0 commented Nov 29, 2024

Ah yes - I think the autoformatter was broken on my machine when I made this PR, so I gave up and crossed my fingers. Now fixed!

@geoffw0 geoffw0 merged commit eeed2c2 into github:redsun82/swift-6 Nov 29, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-change-note-required This PR does not need a change note Swift
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants