Skip to content

馃 Pick PR #52984 (Check for strict subtypes and then ...) into release-5.0 - #53085

Merged
Daniel Rosenwasser (DanielRosenwasser) merged 1 commit into
microsoft:release-5.0from
typescript-bot:pick/52984/release-5.0
Mar 7, 2023
Merged

馃 Pick PR #52984 (Check for strict subtypes and then ...) into release-5.0#53085
Daniel Rosenwasser (DanielRosenwasser) merged 1 commit into
microsoft:release-5.0from
typescript-bot:pick/52984/release-5.0

Conversation

@typescript-bot

Copy link
Copy Markdown
Contributor

This cherry-pick was triggered by a request on #52984
Please review the diff and merge if no changes are unexpected.
You can view the cherry-pick log here.

cc Jake Bailey (@jakebailey)

Component commits:
b3d3ec9 Use strictSubtypeRelation in getNarrowedType and narrow only for pure subtypes

0325f9d Accept new baselines

3df807f First check for strict subtypes, then check for regular subtypes

d737eee Accept new baselines

9b2d602 Add tests

9ea8a55 Accept new baselines

8bb30e2 Add another repro
@DanielRosenwasser

Copy link
Copy Markdown
Member

I tested the top 250 between latest and rc, and that run is over at #53122. I don't see any issues, so I am going to merge.

@DanielRosenwasser
Daniel Rosenwasser (DanielRosenwasser) merged commit 79bdd93 into microsoft:release-5.0 Mar 7, 2023
@DanielRosenwasser

Daniel Rosenwasser (DanielRosenwasser) commented Mar 7, 2023

Copy link
Copy Markdown
Member

Actually, there are two issues that may be notable:

examples/mega-form/tsconfig.json
error TS2589: Type instantiation is excessively deep and possibly infinite.
examples/mega-form/src/App.tsx#L17

boilerplate/tsconfig.json
error TS2345: Argument of type '[never, never]' is not assignable to parameter of type 'never'.
boilerplate/app/navigators/navigationUtilities.ts#L156

But I don't think they're related to this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants