-
Notifications
You must be signed in to change notification settings - Fork 17
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
chore: use maven instead of 'java -jar' to run sample #11
Conversation
@svor done with both your comments. |
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.
Commands work fin on y side.
I just have a problem with the debugger:
@vitaliy-guliy could you please verify if it's a problem with the command that runs the application in not debug mode or maybe it's more general problem and we need another issue for it.
@svor done with both your comments. |
devfile.yaml
Outdated
group: | ||
kind: run | ||
isDefault: true | ||
|
||
- id: run-debug | ||
- id: run-with-mysql-new |
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 run-with-mysql
would be better
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.
Yes, it's definitely better to use command name without new
ending.
Thanks.
Signed-off-by: Vitaliy Gulyy <vgulyy@redhat.com>
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.
Signed-off-by: Vitaliy Gulyy vgulyy@redhat.com
For commands to run sample, it's better to use
mvn spring-boot:run
instead ofjava -jar
Solves eclipse-che/che#20791
To test use
https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/dashboard/#/https://github.com/vitaliy-guliy/java-spring-petclinic/tree/devfilev2&che-editor=che-incubator/che-code/insiders