Repository navigation
DOM update 2023-03-07 - #53133
DOM update 2023-03-07#53133Nathan Shively-Sanders (sandersn) merged 1 commit into
Conversation
Adds many things that are now available in Safari, but were previously Chrome-only. Unlikely to break anybody except those using shim packages from DT, and I'll update those after merging this.
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the parallelized Definitely Typed test suite on this PR at 29914d4. You can monitor the build here. Update: The results are in! |
|
Hey Nathan Shively-Sanders (@sandersn), the results of running the DT tests are ready. Branch only errors:Package: dom-mediacapture-transform Package: dom-screen-wake-lock Package: w3c-css-typed-object-model-level-1 Package: dom-webcodecs Package: use-color-scheme Package: dom-mediacapture-record Package: web-animations-js Package: webappsec-credential-management |
|
Looks like the shims that are broken are:
Plus an $ExpectType in use-color-scheme's tests changes order because a type alias to a union of strings changes order. |
|
TypeScript Bot (@typescript-bot) test this |
|
Since this is additive-only, I doubt there will be any unexpected breaks. I think it's more effective to estimate the size of the break by looking at number of downloads per month for the packages:
Anybody that upgrades to 5.1 will also need to update these packages. That's a lot of downloads put all together. TypeScript Bot (@typescript-bot) test this (I predict that we'll be disappointed with the perf results, since adding this much code is a visible drag on performance.) |
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the perf test suite on this PR at 29914d4. You can monitor the build here. Update: The results are in! |
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the diff-based top-repos suite on this PR at 29914d4. You can monitor the build here. Update: The results are in! |
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the diff-based user code test suite on this PR at 29914d4. You can monitor the build here. Update: The results are in! |
|
Heya Nathan Shively-Sanders (@sandersn), I've started to run the extended test suite on this PR at 29914d4. You can monitor the build here. |
|
Nathan Shively-Sanders (@sandersn) Here are the results of running the user test suite comparing Something interesting changed - please have a look. Details
|
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
user test suite failures: puppeteer has a ts-expect-error where they wanted to use a Chrome-only feature that is now available in Safari. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Nathan Shively-Sanders (@sandersn) Here they are:
CompilerComparison Report - main..53133
System
Hosts
Scenarios
TSServerComparison Report - main..53133
System
Hosts
Scenarios
StartupComparison Report - main..53133
System
Hosts
Scenarios
Developer Information: |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
xstate check time gets 1% better on a change that should not affect checking at all. |
|
Random noise is still possible, I'm afraid :( TypeScript Bot (@typescript-bot) test this |
This comment was marked as outdated.
This comment was marked as outdated.
|
TypeScript Bot (@typescript-bot) test this |
|
Heya Jake Bailey (@jakebailey), I've started to run the extended test suite on this PR at 29914d4. You can monitor the build here. |
|
Nathan Shively-Sanders (@sandersn) Here are the results of running the top-repos suite comparing Everything looks good! |
|
TypeScript Bot (@typescript-bot) test this |
|
Heya Jake Bailey (@jakebailey), I've started to run the extended test suite on this PR at 29914d4. You can monitor the build here. |
|
RWC tests are fine -- no new failures, just some messages about missing even more properties. |
|
Kagami Sascha Rosylight (@saschanaz) Thanks so much for your work in making these types available to Typescript. |
|
Yay, and also thanks to the people who maintain the data in BCD! Queen Vinyl Da.i'gyu-Kazotetsu (@queengooborg) Sam Sneddon (@gsnedders) |
This change was added in microsoft/TypeScript#53133
Adds many things that are now available in Safari, but were previously Chrome-only. Unlikely to break anybody except those using shim packages from DT, and I'll update those after merging this.
Since there is a lot of new stuff here, I wanted to get it into 5.1 as early as possible.