Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Refactor:
webpack-cli
, more intuitiveupdate-notifier
,semver
andterser-webpack-plugin
semver
: not needed at the moment. We use it just for one check. We already specify the engine inside thepackage.json
update-notifier
: it's nice but not essential. Let's save spaceterser-webpack-plugin
: webpack already applies minification when themode
is production. Not neededinquirer
toenquirer
: smaller, less dependencies and same functionality (thank you @rishabh3112 for the suggestion)Did you add tests for your changes?
No
If relevant, did you update the documentation?
Not needed
Summary
The
cli
folder doesn't respect the name insidepackage.json
. That's why I moved it underwebpack-cli
folder. Also, I decided to remove some libraries that are not essential to the CLI and that could save us some space.I also ditched
inquirer
for a more lighter version:enquirer
. It has the same functionalities and is lighter is terms of dependencies and weight.Does this PR introduce a breaking change?
No
Other information