-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Distinguish between Maven and Gradle for hint-text #4899
Conversation
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 suggested we use mvn and gradle wrappers instead. Otherwise, LGTM. Thanks.
} | ||
else { | ||
System.out.println("\nAdd an extension to your project by adding the dependency to your " + | ||
"pom.xml or use `mvn quarkus:add-extension -Dextensions=\"artifactId\"`"); |
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.
"pom.xml or use `mvn quarkus:add-extension -Dextensions=\"artifactId\"`"); | |
"pom.xml or use `./mvnw quarkus:add-extension -Dextensions=\"artifactId\"`"); |
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.
agreed
independent-projects/tools/common/src/main/java/io/quarkus/cli/commands/ListExtensions.java
Outdated
Show resolved
Hide resolved
CI failure looks unrelated, can you try to rebase to latest master and see if that fixes the issue. |
rebased to latest master and integrated your suggested changes |
Cool, let's see if CI is happy this time. Also, sorry to be nitpicking, can we also squash everything the two commits into one. |
Done! |
Can you please rebase on master to pick up a CI fix? |
independent-projects/tools/common/src/main/java/io/quarkus/cli/commands/ListExtensions.java
Outdated
Show resolved
Hide resolved
independent-projects/tools/common/src/main/java/io/quarkus/cli/commands/ListExtensions.java
Outdated
Show resolved
Hide resolved
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.
LGTM
Fixes #4778