-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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 MessageEventTarget missed this parameters #61086
base: main
Are you sure you want to change the base?
DOM update MessageEventTarget missed this parameters #61086
Conversation
@typescript-bot test this |
@sandersn Here are the results of running the user tests with tsc comparing Everything looks good! |
Hey @sandersn, the results of running the DT tests are ready. Everything looks the same! |
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@sandersn Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
matching dom lib generator PR: microsoft/TypeScript-DOM-lib-generator#1894 @typescript-bot test this |
@sandersn Here are the results of running the user tests with tsc comparing Everything looks good! |
Hey @sandersn, the results of running the DT tests are ready. Everything looks the same! |
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@sandersn Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
While trying to update typescript-dom-lib-generator so that it generates the correct types for MessageEventTarget's this parameters, I realised that my manual fix missed a couple. They're not required to make the known problem code in DT/top400 compile, but all the
this
parameters should have consistent types.Of course, the whole thing is so fragile that I want to run tests as I figure out how to make dom-lib-generator generate the correct types, so that I can know if I need an even-more-complex fix.