Skip to content

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

Merged
jamesgeorge007 merged 1 commit into
webpack:nextfrom
snitin315:info
Apr 6, 2020
Merged

feat(webpack-cli): add alias for version#1405
jamesgeorge007 merged 1 commit into
webpack:nextfrom
snitin315:info

Conversation

@snitin315

@snitin315 snitin315 commented Apr 2, 2020

Copy link
Copy Markdown
Member

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
Comment thread test/version/version-multi-args.test.js Outdated
@snitin315

This comment has been minimized.

@alexander-akait

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
Contributor

This seems to be in scope of #1347

@alexander-akait

Copy link
Copy Markdown
Member

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

@jamesgeorge007

Copy link
Copy Markdown
Contributor

Sounds good 👍

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

Copy link
Copy Markdown
Contributor

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