-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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 2023-03-07 #53133
DOM update 2023-03-07 #53133
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.
@typescript-bot run dt |
Heya @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 @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 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 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 @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 @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 @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! |
@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.
@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 test this |
This comment was marked as outdated.
This comment was marked as outdated.
@typescript-bot test this |
Heya @jakebailey, I've started to run the extended test suite on this PR at 29914d4. You can monitor the build here. |
@sandersn Here are the results of running the top-repos suite comparing Everything looks good! |
@typescript-bot test this |
Heya @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. |
@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! @queengooborg @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.