-
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 2023-04-23 #53652
DOM update 2023-04-23 #53652
Conversation
@typescript-bot run dt |
Heya @sandersn, I've started to run the diff-based user code test suite on this PR at 9367197. You can monitor the build here. Update: The results are in! |
@sandersn Here are the results of running the user test suite comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
@typescript-bot perf test this faster |
Heya @DanielRosenwasser, I've started to run the abridged perf test suite on this PR at 9367197. You can monitor the build here. Update: The results are in! |
You'll want to drop the "fixed" from the description as GitHub will close the issue on merge. |
Swapped in a highfalutin' substitute. |
@DanielRosenwasser Here they are:Comparison Report - main..53652
System
Hosts
Scenarios
Developer Information: |
xstate/material-ui getting only 1% slower is a pretty good outcome actually. I guess my optimisation of parsing of comment text paid off. |
Addresses #52956, at least for the DOM part.
I'm scrolling through all 10,000 lines of additions, but so far there are only or two minor changes besides the addition of MDN links to everything.
After seeing how slow jsdoc parsing is, I'm not 100% happy about this, but I think we should (1) run perf tests (2) take this for the beta (3) if the perf tests are horrendous or we think of some other reason, consider reverting the typescript-dom-lib-generator PR.
The most interesting non-comment change is the removal of
type BitrateMode = "constant" | "variable";
in favour of VideoEncoderBitrateMode. A couple of new types/interfaces got added as well.