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

Backport "Fix superType of SuperType" to LTS #18942

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Backport "Fix superType of SuperType" to LTS #18942

merged 2 commits into from
Nov 17, 2023

Conversation

Kordyjan
Copy link
Contributor

Backports #17574 to the LTS branch.

PR submitted by the release tooling.

I am not quite sure abput he previous definition of superType in SuperType.
I believe it's probably needed for something. But it's clearly wrong if
the `supertpe` argument does not have a symbol. We now fall back to the
default `superType = underlying` in this case.

Fixes $17555

[Cherry-picked 810a396]
Base automatically changed from lts-17576 to lts-17571 November 17, 2023 13:49
@Kordyjan
Copy link
Contributor Author

@Kordyjan Kordyjan merged commit f2d5539 into lts-17571 Nov 17, 2023
33 checks passed
@Kordyjan Kordyjan deleted the lts-17574 branch November 17, 2023 14:07
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