Conversation
Overview list of changes still to come.
|
@typescript-bot test it |
|
Hey @jakebailey, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: rx-dom Package: webrtc Package: node Package: dom-mediacapture-record Package: css-font-loading-module Package: dom-webcodecs |
|
@jakebailey Here are the results of running the user tests with tsc comparing Everything looks good! |
|
DT errors: Polyfills that need to be updated:
A user of FontFaceSet that needs to update
A user of MutationEvent that needs a shim now, I guess?
Node has a broken ExpectType:
|
| @@ -450,6 +540,18 @@ interface ElementDefinitionOptions { | |||
| extends?: string; | |||
| } | |||
|
|
|||
| interface EncodedAudioChunkInit { | |||
| data: AllowSharedBufferSource; | |||
| duration?: number; | |||
There was a problem hiding this comment.
Maybe this is generically wrong with the DOM types, but this is seemingly an options bag given to a constructor. Why don't all of these have | undefined?
There was a problem hiding this comment.
I'd have to go read the code but I'm pretty sure it's an oversight that the generator never got updated for the missing/undefined distinction.
|
@jakebailey Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Hm, 26 new errors in the angular benchmark. Maybe that will show in the top tests, but it sounds like I should add that as a user test. |
|
@jakebailey Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
|
Retesting user tests now that I added angular there explicitly: @typescript-bot user test this |
|
@jakebailey Here are the results of running the user tests with tsc comparing Everything looks good! |
|
I'll send PRs for the DT failures later today. The angular result for the most recent user test run says 36 errors before, 36 after. |
CompressionStream.readable is now generic: ReadableStream<Uint8Array>. microsoft/TypeScript#60061
Required by the TS 5.7 PR: microsoft/TypeScript#60061
Required by the TS 5.7 PR: microsoft/TypeScript#60061 Note that this won't pass tests until TS 5.7 is available as a version on DT.
Required by the TS 5.7 PR: microsoft/TypeScript#60061 Note that this won't pass tests until TS 5.7 is available as a version on DT.
Required by the TS 5.7 PR: microsoft/TypeScript#60061
Required by the TS 5.7 PR: microsoft/TypeScript#60061 Note that this won't pass tests until TS 5.7 is available as a version on DT.
This update is almost all additions; the deprecated MutationEvent is removed, and some events have more specific types. I think this is a low-impact update.
prfproperties in AuthenticationExtensionsClientInputs/OutputsbackgroundBlurpropertiesaltitudeAngleandazimuthAnglefragmentDirective,caretPositionFromPoint,currentCSSZoomsource: BufferSourceinstead ofsource: BinaryDatainitialFaces: FontFace[]parameter.writingSuggestions🤖initparameter.Fixes #59704