Skip to content

Swift: QLDoc and test for getCanonicalType#14618

Merged
MathiasVP merged 4 commits intogithub:mainfrom
geoffw0:qldoc
Oct 30, 2023
Merged

Swift: QLDoc and test for getCanonicalType#14618
MathiasVP merged 4 commits intogithub:mainfrom
geoffw0:qldoc

Conversation

@geoffw0
Copy link
Copy Markdown
Contributor

@geoffw0 geoffw0 commented Oct 27, 2023

Add more detailed QLDoc and test for Type.getCanonicalType(). This is a useful method that I've not used until now because, frankly, I didn't really know what it did. Now I do and others can too.

Thanks to @rdmarsh2 for providing the first version of the text for this in #14592 (comment) .

@geoffw0 geoffw0 added no-change-note-required This PR does not need a change note Swift labels Oct 27, 2023
@geoffw0 geoffw0 requested a review from a team as a code owner October 27, 2023 13:44
Comment thread swift/schema.py Outdated
canonical_type: "Type" | desc("""
This is the unique type we get after resolving aliases and desugaring. For example, given
```
typealias MyInt == Int
Copy link
Copy Markdown
Contributor

@MathiasVP MathiasVP Oct 27, 2023

Choose a reason for hiding this comment

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

I don't think == is valid Swift here, is it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Whoops...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@MathiasVP MathiasVP merged commit 4e08ba6 into github:main Oct 30, 2023
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