-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[webpack] Add ProgressPlugin #4530
[webpack] Add ProgressPlugin #4530
Conversation
@@ -10,6 +10,29 @@ declare module 'webpack' { | |||
|
|||
declare type WebpackError = $WebpackError; | |||
|
|||
declare type WebpackLogger = {| | |||
getChildLogger: (arg0: string | (() => string)) => WebpackLogger, | |||
error(...args: any[]): void, |
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.
I thinks spread params should rather use $ReadOnlyArray
, but probably not big deal here.
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.
Great tip! Will start using this going forward
| ProgressHandler; | ||
|
||
declare class ProgressPlugin { | ||
constructor(options?: ProgressPluginArgument): $ElementType< |
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.
$ElementType
, $PropertyType
are deprecated, but this would force to clone/scope to >0.155 that libdef.
Maybe there will be a codemod pass for this one day.
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 other plugins had the same syntax so I just followed. Have fixed them now across the definition
Closes #4528
Other notes: