Skip to content

Conversation

anikethsaha
Copy link
Member

Refactored the code to fix the error of supporting the native plugin in
add-generator. removed the toLowerCase() method, added the logic for
more general plugin name from the input

What kind of change does this PR introduce?

bugfix/refactoring

Did you add tests for your changes?
NA
If relevant, did you update the documentation?
NA
Summary
fix #906
fixed support for adding native plugin in add-generator

Does this PR introduce a breaking change?

No

Other information
NA

refractored the code to fix the error of supporting the native plugin in
add-generator. removed the toLowerCase() method, added the logic for
more general plugin name from the input
@webpack-bot
Copy link

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

@webpack-bot
Copy link

@evenstensberg Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@anshumanv Please review the new changes.

Copy link
Member

@evenstensberg evenstensberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks for your PR!

@evenstensberg evenstensberg merged commit 71a58b6 into webpack:master Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: 🐞 webpack-cli add wont lookup for webpack's native plugin in plugins options
5 participants