-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Do not infer to mapped types from implied index signatures created by rest bindings #52632
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Do not infer to mapped types from implied index signatures created by rest bindings #52632
Conversation
@typescript-bot test this |
Heya @jakebailey, I've started to run the diff-based user code test suite on this PR at 8470be3. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the tarball bundle task on this PR at 8470be3. You can monitor the build here. |
Heya @jakebailey, I've started to run the diff-based top-repos suite (tsserver) on this PR at 8470be3. 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 8470be3. You can monitor the build here. |
Heya @jakebailey, I've started to run the perf test suite on this PR at 8470be3. 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 (tsserver) on this PR at 8470be3. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the diff-based top-repos suite on this PR at 8470be3. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the extended test suite on this PR at 8470be3. You can monitor the build here. |
Hey @jakebailey, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build and an npm module you can use via |
@jakebailey Here are the results of running the user test suite comparing Everything looks good! |
1 similar comment
@jakebailey Here are the results of running the user test suite comparing Everything looks good! |
Heya @jakebailey, I've run the RWC suite on this PR - assuming you're on the TS core team, you can view the resulting diff here. |
@jakebailey Here they are:
CompilerComparison Report - main..52632
System
Hosts
Scenarios
TSServerComparison Report - main..52632
System
Hosts
Scenarios
StartupComparison Report - main..52632
System
Hosts
Scenarios
Developer Information: |
@jakebailey Here are the results of running the top-repos suite comparing Everything looks good! |
@jakebailey Here are the results of running the top-repos suite comparing Something interesting changed - please have a look. Detailsapache/echarts
|
@typescript-bot test top100 |
Heya @weswigham, I've started to run the parallelized Definitely Typed test suite on this PR at 8470be3. You can monitor the build here. Update: The results are in! |
Heya @weswigham, I've started to run the diff-based top-repos suite on this PR at 8470be3. You can monitor the build here. |
Hey @weswigham, it looks like the DT test run failed. Please check the log for more details. |
Hm, looks like there's a conflict with main? |
Looks like it, yep. |
…om-implied-index-infos-to-mapped-types
resolved the said conflict, @jakebailey could you re-run the requested jobs? |
@typescript-bot test this |
Heya @jakebailey, I've started to run the diff-based top-repos suite on this PR at af648b8. 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 af648b8. 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 af648b8. 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 af648b8. You can monitor the build here. Update: The results are in! |
Heya @jakebailey, I've started to run the extended test suite on this PR at af648b8. You can monitor the build here. |
@jakebailey Here are the results of running the user test suite comparing Everything looks good! |
@jakebailey Here they are:Comparison Report - main..52632
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. |
fixes #52629
cc @andrewbranch , it might be interesting to you given your work on #49086