-
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
feat(25758): Computed properties no throws if there is a duplicate #49848
Conversation
@typescript-bot test top100 |
Heya @RyanCavanaugh, I've started to run the diff-based top-repos suite on this PR at 69b4751. You can monitor the build here. Update: The results are in! |
@RyanCavanaugh Here are the results of running the top-repos suite comparing Everything looks good! |
@typescript-bot perf test this |
Heya @RyanCavanaugh, I'm starting to run the perf test suite on this PR at 69b4751. Hold tight - I'll update this comment with the log link once the build has been queued. |
@typescript-bot perf test this but maybe work this time? |
Heya @jakebailey, I've started to run the perf test suite on this PR at 69b4751. You can monitor the build here. Update: The results are in! |
@jakebailey Here they are:
CompilerComparison Report - main..49848
System
Hosts
Scenarios
TSServerComparison Report - main..49848
System
Hosts
Scenarios
Developer Information: |
@typescript-bot test this |
Heya @jakebailey, I've started to run the extended test suite on this PR at 80dccc9. You can monitor the build here. |
Heya @jakebailey, I've started to run the diff-based top-repos suite on this PR at 80dccc9. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the diff-based user code test suite on this PR at 80dccc9. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the parallelized Definitely Typed test suite on this PR at 80dccc9. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the abridged perf test suite on this PR at 80dccc9. You can monitor the build here. Update: The results are in! |
@jakebailey Here are the results of running the user test suite comparing Everything looks good! |
@jakebailey Here they are:Comparison Report - main..49848
System
Hosts
Scenarios
Developer Information: |
@jakebailey Here are the results of running the top-repos suite comparing Everything looks good! |
Hey @jakebailey, the results of running the DT tests are ready. Errors that changed:Package: dom-mediacapture-record
Branch error:
|
Checking https://github.com/microsoft/TypeScript/pull/49848/files?w=1 this feels okay to me. This has been in review for a while. Are there any other concerns here? |
@weswigham Thanks |
Fixes #25758
Fixes #43186
Fixes #49666