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

feat(webpack-cli): add alias for version #1405

Merged
merged 1 commit into from
Apr 6, 2020

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Apr 2, 2020

What kind of change does this PR introduce?

Feature. added alias for version.

Did you add tests for your changes?
YES

If relevant, did you update the documentation?
YES

Summary

Fixes #1368

Screenshot from 2020-04-04 20-55-44

Does this PR introduce a breaking change?

No
Other information
NA

@snitin315 snitin315 requested a review from a team as a code owner April 2, 2020 12:22
alexander-akait
alexander-akait previously approved these changes Apr 2, 2020
alexander-akait
alexander-akait previously approved these changes Apr 2, 2020
@snitin315

This comment has been minimized.

@alexander-akait
Copy link
Member

@snitin315 Please open an issue about it

@snitin315 snitin315 force-pushed the info branch 3 times, most recently from 02e7a70 to e0275f2 Compare April 3, 2020 13:45
@anshumanv
Copy link
Member

This should be handled after #1412 or maybe make your changes here and update tests accordingly.

@snitin315 snitin315 changed the title tests(webpack-cli): add more cases in version-multi-args-test feat(webpack-cli): add alias for version Apr 4, 2020
@snitin315
Copy link
Member Author

@anshumanv I have updated the PR for a standalone change and updated the test accordingly.

I will tackle #1412 in a separate PR.

alexander-akait
alexander-akait previously approved these changes Apr 6, 2020
anshumanv
anshumanv previously approved these changes Apr 6, 2020
jamesgeorge007
jamesgeorge007 previously approved these changes Apr 6, 2020
@jamesgeorge007
Copy link
Member

This seems to be in scope of #1347

@alexander-akait
Copy link
Member

@jamesgeorge007 we can implement it here, and migrate on commander late

@jamesgeorge007
Copy link
Member

Sounds good 👍

docs: add alias for version
@jamesgeorge007 jamesgeorge007 merged commit 6b9461e into webpack:next Apr 6, 2020
@jamesgeorge007
Copy link
Member

Thanks

@snitin315 snitin315 deleted the info branch April 7, 2020 03:04
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.

Feature: add alias for version flag
5 participants