Repository navigation
Make substitution types even if the substitution base isnt a type variable - #37348
Conversation
|
TypeScript Bot (@typescript-bot) user test this |
|
Heya Wesley Wigham (@weswigham), I've started to run the parallelized community code test suite on this PR at 7772caf. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the extended test suite on this PR at 7772caf. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the parallelized Definitely Typed test suite on this PR at 7772caf. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the perf test suite on this PR at 7772caf. You can monitor the build here. Update: The results are in! |
|
The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master. |
|
Wesley Wigham (@weswigham) Here they are:Comparison Report - master..37348
System
Hosts
Scenarios
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Tests and benches looked good; updates the fix to be broader, rerunning tests: TypeScript Bot (@typescript-bot) user test this |
|
Heya Wesley Wigham (@weswigham), I've started to run the perf test suite on this PR at c7f54d5. You can monitor the build here. Update: The results are in! |
|
Heya Wesley Wigham (@weswigham), I've started to run the parallelized community code test suite on this PR at c7f54d5. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the extended test suite on this PR at c7f54d5. You can monitor the build here. |
|
Heya Wesley Wigham (@weswigham), I've started to run the parallelized Definitely Typed test suite on this PR at c7f54d5. You can monitor the build here. |
|
Wesley Wigham (@weswigham) Here they are:Comparison Report - master..37348
System
Hosts
Scenarios
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Perf looks fine, all test baselines look fine. This change seems good; Anders Hejlsberg (@ahejlsberg) you wanna weigh in on this? It's essentially just generalizing substitutions to represent arbitrary type-space "narrowings", rather than just substitutions on generics. |
Fixes #36135