-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Cache widened types #31586
Cache widened types #31586
Conversation
@typescript-bot test this |
Heya @weswigham, I've started to run the extended test suite on this PR at 440cc7f. You can monitor the build here. It should now contribute to this PR's status checks. |
Heya @weswigham, I've started to run the community code test suite on this PR at 440cc7f. You can monitor the build here. It should now contribute to this PR's status checks. |
Heya @weswigham, I've started to run the parallelized Definitely Typed test suite on this PR at 440cc7f. You can monitor the build here. It should now contribute to this PR's status checks. |
@typescript-bot test this now that that lint is fixed 🤦♂ |
Heya @weswigham, I've started to run the extended test suite on this PR at 2a25fc7. You can monitor the build here. It should now contribute to this PR's status checks. |
DT passes, as expected 🌞 |
@RyanCavanaugh All the test suites are gucci |
@typescript-bot perf test this - this should make for some perf gains, even in older style code (but moreso in JS). |
Heya @weswigham, I've started to run the perf test suite on this PR at 2a25fc7. You can monitor the build here. It should now contribute to this PR's status checks. Update: The results are in! |
@weswigham Here they are:Comparison Report - master..31586
System
Hosts
Scenarios
|
@weswigham I think get an infinite loop. I have a repository+commit link in that issue with a repro (not minimal) |
Considering this wasn't published in a nighly until around... Now? I doubt it? Unless you mean the new nightly fixes your problem and you think this fixed it. (In which case: I unno) |
Fixes #29902
Probably fixes #29564 (doesn't have a full repro)
Notably does not fix #29597 - that has some more stuff going on - chart.min.js is uuuuuuge.