-
Notifications
You must be signed in to change notification settings - Fork 12.8k
DOM Update 2025-05-02 #61647
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
base: main
Are you sure you want to change the base?
DOM Update 2025-05-02 #61647
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the DOM types to incorporate changes from the referenced TypeScript-DOM-lib-generator pull request, while ensuring that related error messages and contextual type instantiations continue to behave as expected.
- Updates baseline errors for instantiateContextualTypes tests
- Covers new contextual typing scenarios with generic components and reducer patterns
Files not reviewed (6)
- tests/baselines/reference/collisionThisExpressionAndLocalVarInLambda.types: Language not supported
- tests/baselines/reference/globalThisBlockscopedProperties.types: Language not supported
- tests/baselines/reference/instantiateContextualTypes.types: Language not supported
- tests/baselines/reference/noCollisionThisExpressionAndLocalVarInLambda.types: Language not supported
- tests/baselines/reference/underscoreTest1.types: Language not supported
- tests/baselines/reference/unicodeIdentifierName2.types: Language not supported
@typescript-bot test this |
Hey @rbuckton, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: jquery.fancytree
Package: w3c-image-capture
Package: summernote
Package: node
Package: ckeditor4
Package: dropboxjs
Package: microsoft-ajax
Package: photoswipe
Package: hashset
Package: codemirror
Package: signals
Package: chrome
Package: react-swipe
Package: snapsvg
Package: jquery/v1
Package: jquery/v2
Package: jquery
Package: jquery-touchswipe
Package: phonegap
Package: three
Package: less
Package: lscache
Package: switchery
|
@rbuckton Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Something interesting changed - please have a look. Details
|
@rbuckton Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
A large number of failures seem to be due to the signature of |
@rbuckton Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
FYI @saschanaz @Bashamega |
@typescript-bot run dt |
Starting jobs; this comment will be updated as builds start and complete.
|
@typescript-bot run dt |
Any idea what the failure looks like? |
Err, okay, |
|
Actually I'm talking about https://github.com/microsoft/TypeScript/actions/runs/14803024546/job/41565917783?pr=61647, but that's just an internal test? |
Hey @rbuckton, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: w3c-image-capture
Package: node
Package: three
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides ArrayBufferLike, it's almost all additions. Plus tonnes of comment changes.
Should we rollback that PR @saschanaz |
Should "click" and several other event types not also be changed from |
See #61647 (comment). A large percentage of the DT tests failures are due to the signature of |
1fd8451
to
846c7dc
Compare
|
|
Ah, the "is not assignable to parameter of type 'string'" errors. Err, still looks like something TS test specific, but perhaps also maybe an early smoke of something bigger. I'm open to backout. |
There are similar errors in the user tests, though not as frequent. |
@typescript-bot run dt |
@rbuckton Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Something interesting changed - please have a look. Details
|
Hey @rbuckton, the results of running the DT tests are ready. Everything looks the same! |
@rbuckton Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
This updates the DOM types to include changes from microsoft/TypeScript-DOM-lib-generator#1944, among others.
Fixes #61480