Skip to content

Conversation

@dummdidumm
Copy link
Member

Fixes a overzealous check introduced through #1218

Fixes a overzealous check introduced through sveltejs#1218
Copy link
Member

@jasonlyu123 jasonlyu123 left a comment

Choose a reason for hiding this comment

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

Looks good. Not quite certain how it works but it seems like any is the only one that takes priority over unknown in here.

@dummdidumm
Copy link
Member Author

Yeah I'm a little confused myself about this, almost like TS takes the results of the extends clause into account when deciding what to resolve to. Thanks for having a look!

@dummdidumm dummdidumm merged commit b1b0484 into sveltejs:master Nov 17, 2021
@dummdidumm dummdidumm deleted the fix-each branch November 17, 2021 07:44
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