DOM update - #62730
Conversation
|
TypeScript Bot (@typescript-bot) user test this |
There was a problem hiding this comment.
Pull Request Overview
This PR updates test baseline files that capture expected compiler output. The changes reflect updates to performance metrics and type information that the TypeScript compiler generates.
- Updates Symbol count threshold from 50,000 to 100,000 in JSX test baselines
- Adds "oncommand" to the globalThis type union and adjusts line numbers accordingly
Reviewed Changes
Copilot reviewed 2 out of 6 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/baselines/reference/jsxIntrinsicElementsCompatability.types | Updates expected Symbol count metric from 50,000 to 100,000 |
| tests/baselines/reference/globalThisBlockscopedProperties.types | Adds "oncommand" event handler to the globalThis type union and adjusts line numbers |
|
Hey Jake Bailey (@jakebailey), the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: serviceworker-webpack-plugin |
|
Jake Bailey (@jakebailey) Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
|
Jake Bailey (@jakebailey) Here are the results of running the top 800 repos with tsc comparing Something interesting changed - please have a look. Details
|
|
René (@Renegade334) Do these differences seem correct to you? I think these are a result of the change you just made. |
|
The top800 ones? Those do look correct – these are essentially the exact same |
|
TypeScript Bot (@typescript-bot) test it |
|
Hey Jake Bailey (@jakebailey), the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: dom-navigation Package: serviceworker-webpack-plugin |
|
Jake Bailey (@jakebailey) Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
|
Jake Bailey (@jakebailey) Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Jake Bailey (@jakebailey) Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
|
Hm, I guess the generated |
It's the new Navigation API that @types/dom-navigation is clashing with. |
f410e03 to
bcf9757
Compare
bcf9757 to
af64efe
Compare
|
TypeScript Bot (@typescript-bot) test it |
|
Jake Bailey (@jakebailey) Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
|
Hey Jake Bailey (@jakebailey), the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: deno Package: dom-navigation Package: serviceworker-webpack-plugin |
|
The GPU types get bigger, and so more conflicts... |
|
Jake Bailey (@jakebailey) Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Jake Bailey (@jakebailey) Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
|
Going to take these as-is for now. We update DOM before the RC too, so if something's still missing it's fixable. |
Pulling in the latest since a bunch changed.