Skip to content
This repository was archived by the owner on Dec 18, 2021. It is now read-only.

Add "--wrapper-version" argument to show wrapper version#48

Merged
mosabua merged 1 commit intotakari:masterfrom
ttddyy:wrapper-version
Apr 28, 2017
Merged

Add "--wrapper-version" argument to show wrapper version#48
mosabua merged 1 commit intotakari:masterfrom
ttddyy:wrapper-version

Conversation

@ttddyy
Copy link
Copy Markdown

@ttddyy ttddyy commented Apr 28, 2017

Adding special handling to --wrapper-version argument.

Since all maven commands are delegated to plexus launcher, intercept --wrapper-version argument, printout wrapper version, then exit the execution before calling plexus launcher.

Relates to #44

@mosabua
Copy link
Copy Markdown
Member

mosabua commented Apr 28, 2017

NIce .. will review, test and merge soon.

@mosabua
Copy link
Copy Markdown
Member

mosabua commented Apr 28, 2017

Thinking about it.. how about instead of just adding another option that no one knows about .. spit out the wrapper version into the log with each invocation.. its one line .. and it would help and no one would need to know any option..

@ttddyy
Copy link
Copy Markdown
Author

ttddyy commented Apr 28, 2017

sounds good to me.
updated the PR to print out wrapper version at every invocation instead of new option.

@mosabua mosabua merged commit e03a173 into takari:master Apr 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants