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

refactor(webpack-cli): remove --no-mode flag #1503

Merged
merged 1 commit into from
Apr 29, 2020

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Apr 27, 2020

What kind of change does this PR introduce?

Follow up #1481 .
remove --no-mode.

Did you add tests for your changes?
Removed no-mode tests.

If relevant, did you update the documentation?
Yes

Summary

removed --no-mode because it is same as --mode none and Negated flags makes more sense with boolean args.

Does this PR introduce a breaking change?
Maybe

Other information

@snitin315 snitin315 requested a review from a team as a code owner April 27, 2020 16:37
@alexander-akait
Copy link
Member

/cc @Loonride

Signed-off-by: Nitin Kumar <snitin315@gmail.com>
@alexander-akait alexander-akait merged commit dfb9a91 into webpack:next Apr 29, 2020
@alexander-akait
Copy link
Member

Thanks!

@snitin315 snitin315 deleted the chore/no-mode branch April 29, 2020 10:32
@snitin315 snitin315 changed the title chore(webpack-cli): remove --no-mode flag refactor(webpack-cli): remove --no-mode flag Apr 29, 2020
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.

4 participants