-
Notifications
You must be signed in to change notification settings - Fork 12.8k
🤖 User test baselines have changed #30832
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
🤖 User test baselines have changed #30832
Conversation
6c3c025
to
923c381
Compare
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.
All the changes are results of DOM updates.
I fixed the new error in webpack, but it'll be a few days before it ships. The other errors are OK to accept.
Exit Code: 1 | ||
Standard output: | ||
../../../../../built/local/lib.dom.d.ts(17566,19): error TS2451: Cannot redeclare block-scoped variable 'WebAssembly'. | ||
declarations.d.ts(258,15): error TS2451: Cannot redeclare block-scoped variable 'WebAssembly'. |
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.
We have wEbbasSeMBly now~~~
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.
Fixed by webpack/webpack#9022
@@ -47,6 +47,8 @@ lib/Page.js(112,33): error TS2345: Argument of type 'CDPSession' is not assignab | |||
lib/Page.js(189,15): error TS2503: Cannot find namespace 'Protocol'. | |||
lib/Page.js(284,15): error TS2503: Cannot find namespace 'Protocol'. | |||
lib/Page.js(369,20): error TS2503: Cannot find namespace 'Protocol'. | |||
lib/Page.js(432,7): error TS2740: Type '(...args: any[]) => Promise<any>' is missing the following properties from type 'Window': Blob, TextDecoder, TextEncoder, URL, and 231 more. | |||
lib/Page.js(442,9): error TS2349: Cannot invoke an expression whose type lacks a call signature. Type 'Window' has no compatible call signatures. |
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.
numeric index signature means that window[x] : Window
where x: any
. And the arrow in question is not assignable to Window.
@@ -34,6 +34,8 @@ node_modules/chrome-devtools-frontend/front_end/Runtime.js(693,7): error TS2322: | |||
node_modules/chrome-devtools-frontend/front_end/Runtime.js(705,5): error TS2322: Type 'Promise<boolean>' is not assignable to type 'Promise<undefined>'. | |||
node_modules/chrome-devtools-frontend/front_end/Runtime.js(715,7): error TS2322: Type 'Promise<void>' is not assignable to type 'Promise<undefined>'. | |||
node_modules/chrome-devtools-frontend/front_end/Runtime.js(729,7): error TS2322: Type 'Promise<void>' is not assignable to type 'Promise<undefined>'. | |||
node_modules/chrome-devtools-frontend/front_end/Runtime.js(746,5): error TS2322: Type 'Window | {}' is not assignable to type 'Window'. |
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.
similar problem to puppeteer, except that the assignment almost works.
@@ -3354,7 +3356,8 @@ node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(26,46): error T | |||
node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(44,23): error TS2322: Type 'number' is not assignable to type 'RegExpMatchArray'. | |||
node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(45,23): error TS2365: Operator '>=' cannot be applied to types 'RegExpMatchArray' and 'number'. | |||
node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(47,76): error TS2365: Operator '<' cannot be applied to types 'RegExpMatchArray' and 'number'. | |||
node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(94,25): error TS2339: Property 'createTextRange' does not exist on type 'HTMLElement'. | |||
node_modules/chrome-devtools-frontend/front_end/cm/codemirror.js(94,25): error TS2339: Property 'createTextRange' does not exist on type 'HTMLBodyElement | HTMLFrameSetElement'. |
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.
better type from DOM
Please review the diff and merge if no changes are unexpected.
You can view the build log here.
cc @weswigham @sandersn @RyanCavanaugh