Skip to content
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

fix(webpack-cli): throw error for invalid args #1462

Merged
merged 1 commit into from
Apr 30, 2020

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Apr 15, 2020

What kind of change does this PR introduce?
Bug Fix

Fixes #1412

Did you add tests for your changes?
Yes

If relevant, did you update the documentation?
NA

Summary

Now cli throws an error if invalid args are passed with version or help.

help

Does this PR introduce a breaking change?

Maybe

Other information

@snitin315 snitin315 requested a review from a team as a code owner April 15, 2020 04:11
Copy link
Member

@rishabh3112 rishabh3112 left a comment

Choose a reason for hiding this comment

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

This should be changed for help command in context to DX imo.
We should display help anyway (as user wants help) and a warning should be displayed for passing invalid option.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Read comments

yarn.lock Outdated Show resolved Hide resolved
packages/webpack-cli/lib/groups/HelpGroup.js Outdated Show resolved Hide resolved
@snitin315 snitin315 force-pushed the fix/help-group branch 2 times, most recently from 13d9067 to 8fbf100 Compare April 15, 2020 15:26
@snitin315 snitin315 force-pushed the fix/help-group branch 3 times, most recently from c8ce7aa to ffc896b Compare April 16, 2020 11:49
Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

/cc @webpack/cli-team

@snitin315 snitin315 force-pushed the fix/help-group branch 3 times, most recently from a35ad8c to a346435 Compare April 26, 2020 07:07
@snitin315
Copy link
Member Author

Friendly ping @rishabh3112

rishabh3112
rishabh3112 previously approved these changes Apr 29, 2020
@snitin315
Copy link
Member Author

CI is green 😄

@snitin315
Copy link
Member Author

/cc @jamesgeorge007

@webpack-bot
Copy link

@snitin315 Thanks for your update.

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

@evilebottnawi Please review the new changes.

Signed-off-by: Nitin Kumar <snitin315@gmail.com>
@snitin315
Copy link
Member Author

/cc @rishabh3112 @jamesgeorge007

@rishabh3112 rishabh3112 merged commit 25b3e04 into webpack:next Apr 30, 2020
@snitin315 snitin315 changed the title fix(webpack-cli): throw error for invalid args with HelpGroup fix(webpack-cli): throw error for invalid args Apr 30, 2020
@snitin315 snitin315 deleted the fix/help-group branch April 30, 2020 11:46
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: version flag doesn't care about other arguments
5 participants