Partially disable inference recursion tracking changes#40256
Partially disable inference recursion tracking changes#40256ahejlsberg wants to merge 3 commits intomasterfrom
Conversation
typescript-bot
added
the
For Uncommitted Bug
|
@typescript-bot test this |
|
Heya @ahejlsberg, I've started to run the extended test suite on this PR at e9b8606. You can monitor the build here. |
|
@typescript-bot test this |
|
Heya @ahejlsberg, I've started to run the extended test suite on this PR at 0a506ac. You can monitor the build here. |
|
@weswigham Hmm, the RWC suites are failing even when I back out the changes that could potentially consume more memory by generating more work. I'm pretty sure there's nothing left in the recursive conditional types PR that could be the cause. Wonder if something else could be? |
|
@weswigham Also, here I ran the RWC tests before merging the PR and I'm pretty certain they passed (sadly that build has now gone away). The fact that none of this repros locally of course makes it extra hard to track down. |
|
RWC should be fixed now~ |
|
No idea how a change merged on the timeline after your change was merged somehow affected your change's RWC run, though. |
3 participants
Experiment to see if partially disabling inference recursion tracking changes from recursive conditional types PR resolves the OOM we're seeing in the RWC test suites.