-
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
DOM update 2025/01/29 #61073
DOM update 2025/01/29 #61073
Conversation
This is going into nightly to make sure that microsoft/TypeScript-DOM-lib-generator#1598 is goodto ship for the RC.
@typescript-bot test this |
Hey @sandersn, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: rusha
Package: dom-view-transitions
Errors that changed between main and the branch:Package: dom-webcodecs
Branch error:
|
@typescript-bot user test this |
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@jakebailey Here are the results of running the user tests with tsc comparing Everything looks good! |
@sandersn Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
No new failures from this update, which means that it's good to merge. However, rusha and vscode are failing with the MessagePort-related error that I thought was fixed. I need to see what's going on there. |
See if it fixes DT/vscode compilation
@typescript-bot test this |
@sandersn Here are the results of running the user tests with tsc comparing Everything looks good! |
Hey @sandersn, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: rusha
Package: dom-view-transitions
Errors that changed between main and the branch:Package: dom-webcodecs
Branch error:
|
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@sandersn Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
Re-adding the members of WorkerEventMap didn't help. We should merge this PR after I revert that commit, and I'll figure out the problem separately. |
This reverts commit 1a3d836.
@typescript-bot test this |
@sandersn, the perf run you requested failed. You can check the log here. |
Hey @sandersn, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: dom-view-transitions
Errors that changed between main and the branch:Package: dom-webcodecs
Branch error:
|
rusha is fixed, which is indicative that vscode will also be fixed when I can get a run of it. |
@typescript-bot test top400 |
@jakebailey Here are the results of running the user tests with tsc comparing Everything looks good! |
@jakebailey Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@jakebailey Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
OK, I think everything is ready here, although I need to go fix the output of DOM-lib-generator to be |
I want this into nightly soon to make sure that
microsoft/TypeScript-DOM-lib-generator#1598 is good to ship for the RC. It changes return types of 5 functions from
Promise<undefined>
toPromise<void>
. This should improve compatibility since it's the way the rest of the DOM is, but it's also been long enough that I want to check to make sure.