-
Notifications
You must be signed in to change notification settings - Fork 86
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 for WFMP-235, Introduce a start-jar goal #461
Conversation
I'm on PTO until the 2nd, but I want to think about this. I'll give it a review once I'm back if I don't get bored and review it sooner. :) |
init(); | ||
|
||
@Override | ||
protected Path getServer() throws MojoExecutionException, MojoFailureException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should call this getServerHome()
or getJBossHome()
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done thanks.
* The purpose of this goal is to start a WildFly Application Server packaged as a Bootable JAR for testing during the maven | ||
* lifecycle. | ||
*/ | ||
@Mojo(name = "start-bootable", requiresDependencyResolution = ResolutionScope.RUNTIME) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm just thinking a bit out loud here. Should the goal name be start-jar
? Just thinking of something short to type is all, but I'm not sure I like start-jar
:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I much prefer start-jar, start-bootable is also a bit ambiguous, all server boot... So changed to start-jar.
Issue: https://issues.redhat.com/browse/WFMP-235