-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Revert "Defer type comparability check for assertions" #53879
Conversation
This reverts commit 8947825.
@typescript-bot perf test this |
Heya @DanielRosenwasser, I've started to run the perf test suite on this PR at dd00a34. You can monitor the build here. Update: The results are in! |
@typescript-bot user test this |
Heya @armanio123, I've started to run the diff-based user code test suite on this PR at dd00a34. You can monitor the build here. Update: The results are in! |
@armanio123 Here are the results of running the user test suite comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
@DanielRosenwasser Here they are:
CompilerComparison Report - main..53879
System
Hosts
Scenarios
TSServerComparison Report - main..53879
System
Hosts
Scenarios
StartupComparison Report - main..53879
System
Hosts
Scenarios
Developer Information: |
Closing in favor of #54224 |
Do not merge - experiment to revert #53261