Update @webref/idl to version 3.82.1 - #2535
Conversation
|
Thanks for the PR! This section of the codebase is owned by Kagami Sascha Rosylight (@saschanaz) - if they write a comment saying "LGTM" then it will be merged. |
| declare var ProcessingInstruction: { | ||
| prototype: ProcessingInstruction; | ||
| new(): ProcessingInstruction; | ||
| new(target: string, data?: string): ProcessingInstruction; |
There was a problem hiding this comment.
This is not supported by anyone I believe, we'll have to also remove this in KDL.
There was a problem hiding this comment.
I have fixed it and added support for it in kdl
…s multiple TypeScript definition files and update related handling in patches.
| optional?: boolean; | ||
| variadic?: boolean; | ||
| } | ||
| export type Param = |
There was a problem hiding this comment.
Can you do it without changing types.ts? Patching should not change base types unless the type is already wrong.
…ting Param interface for improved flexibility in method signatures.
| ) { | ||
| params.push(param.name); | ||
| } else { | ||
| params.push(param); |
There was a problem hiding this comment.
It's better, but still kinda sad. I wonder it would be better to somehow treat { name: "foo" } with no other fields as foo on prune() level. If that's possible then we won't need this workaround for any future array items either. Thoughts?
…es.ts for improved type safety and clarity.
…d maintainability.
|
I have improved the logic Kagami Sascha Rosylight (@saschanaz) |
Kagami Sascha Rosylight (saschanaz)
left a comment
There was a problem hiding this comment.
Thanks! I'll need to double check other changes, but the logic patch looks really good 🙏🏻👍🏻🎉
| Object.keys(item).length === 1 && | ||
| Object.prototype.hasOwnProperty.call(item, "name") && | ||
| typeof (item as Record<string, unknown>).name === "string", | ||
| ) |
There was a problem hiding this comment.
Can you extract this lambda in every() into a separate function so that it becomes obj.every(isObjectWithOnlyName) (name free to change)?
There was a problem hiding this comment.
Thanks for the learning, I have updated it
Introduced isObjectWithOnlyName function to streamline checks for objects with a single 'name' property. Updated convertForRemovals to utilize this new helper for improved readability and maintainability.
| @@ -0,0 +1,5 @@ | |||
| removals { | |||
| dictionary WheelEventInit { | |||
| member momentum // Blink only as of 2025-08 | |||
There was a problem hiding this comment.
2025 or 2026? 👀
There was a problem hiding this comment.
LOL, this was a typo, updated
|
Now about the compiler test failure. Jake Bailey (@jakebailey), this is the first time I've seen this test failing. I don't understand why it fails because no line change happened, and the test file looks very simple. what do we do? Ignore it? |
Split #2495