| commit | 269e9ae3ebd8d9c79fa42e19a8674d45afd8cc9f | [log] [tgz] |
|---|---|---|
| author | Tim van der Lippe <[email protected]> | Tue Dec 14 13:33:28 2021 |
| committer | Devtools-frontend LUCI CQ <devtools-frontend-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Dec 14 14:27:48 2021 |
| tree | b47cf834f266c2a62ee3b0b2cc8cf296b350c64e | |
| parent | 338debd13558a8dc5553be666394f561bbc741a2 [diff] [blame] |
Replace any in return values for void This applies the following regex to the codebase: => any and replaces it with => void All obsolete typescript-eslint disable rules have been removed as well. [email protected] Bug: 1172300 Change-Id: I98dc2153f539c68853ed4e9cba8503a4b69763ab Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/3338347 Reviewed-by: Simon Zünd <[email protected]> Reviewed-by: Tim Van der Lippe <[email protected]> Commit-Queue: Tim Van der Lippe <[email protected]> Auto-Submit: Tim Van der Lippe <[email protected]>
diff --git a/front_end/models/extensions/ExtensionAPI.ts b/front_end/models/extensions/ExtensionAPI.ts index ab2ad02..8d62dbe 100644 --- a/front_end/models/extensions/ExtensionAPI.ts +++ b/front_end/models/extensions/ExtensionAPI.ts
@@ -304,8 +304,9 @@ // We cannot use the stronger `unknown` type in place of `any` in the following type definition. The type is used as // the right-hand side of `extends` in a few places, which doesn't narrow `unknown`. Without narrowing, overload // resolution and meaningful type inference of arguments break, for example. + // TODO(crbug.com/1172300) Ignored during the jsdoc to ts migration // eslint-disable-next-line @typescript-eslint/no-explicit-any - export type Callable = (...args: any) => any; + export type Callable = (...args: any) => void; export interface EventSink<ListenerT extends Callable> extends PublicAPI.Chrome.DevTools.EventSink<ListenerT> { _type: string;