Conversation
|
@typescript-bot test this |
|
Heya @sandersn, I've started to run the diff-based user code test suite on this PR at 6cdba31. 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 6cdba31. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the parallelized Definitely Typed test suite on this PR at 6cdba31. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the perf test suite on this PR at 6cdba31. 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 (tsserver) on this PR at 6cdba31. You can monitor the build here. Update: The results are in! |
|
Heya @sandersn, I've started to run the diff-based top-repos suite (tsserver) on this PR at 6cdba31. You can monitor the build here. Update: The results are in! |
|
@sandersn Here are the results of running the user test suite comparing Everything looks good! |
|
@sandersn Here are the results of running the user test suite comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Something interesting changed - please have a look. Details
|
|
@sandersn Here they are:
CompilerComparison Report - main..54725
System
Hosts
Scenarios
TSServerComparison Report - main..54725
System
Hosts
Scenarios
StartupComparison Report - main..54725
System
Hosts
Scenarios
Developer Information: |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@sandersn Here are the results of running the top-repos suite comparing Everything looks good! |
|
Hey @sandersn, it looks like the DT test run failed. Please check the log for more details. |
|
@sandersn Here are the results of running the top-repos suite comparing Everything looks good! |
|
Perf looks OK, just a little more memory used than before. |
|
@sandersn this doesn't include the |
| /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/ScreenOrientation/type) */ | ||
| readonly type: OrientationType; | ||
| /** [MDN Reference](https://developer.mozilla.org/docs/Web/API/ScreenOrientation/lock) */ | ||
| lock(orientation: OrientationLockType): Promise<void>; |
There was a problem hiding this comment.
Is this change correct? It seems like this shouldn't be removed
There was a problem hiding this comment.
DOM update 2023/06/21
canParseto URL.AutoFillgenerated by a number of template type literals. Used for manyautocompleteproperties.sizeanddelete/hasadds optional second parameter.Globalinterface adds a type parameterT extends ValueType = ValueType, used to typevalue/valueOf. (type ValueType = "anyfunc" | "externref" | "f32" | "f64" | "i32" | "i64" | "v128")The only suspicious changes I see are (2) and (7); the former because it might be slow, the latter because I don't understand what
Globalis for, and adding a type parameter in the DOM has gone badly in the past.Fixes #54280