Skip to content

Refine the command to create Java project #252

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

Merged
merged 2 commits into from
Apr 20, 2020
Merged

Refine the command to create Java project #252

merged 2 commits into from
Apr 20, 2020

Conversation

jdneo
Copy link
Member

@jdneo jdneo commented Apr 13, 2020

fix #249
fix #206
fix #199

@jdneo jdneo requested a review from testforstephen April 13, 2020 07:26
@jdneo jdneo requested a review from Eskibear April 13, 2020 07:27
@jdneo jdneo added this to the 0.9.1 milestone Apr 13, 2020
@jdneo jdneo force-pushed the cs/create-project branch from 0013de3 to e95d6e6 Compare April 16, 2020 08:23
Signed-off-by: Sheng Chen <sheche@microsoft.com>
@jdneo jdneo force-pushed the cs/create-project branch from e95d6e6 to 7cca3bb Compare April 16, 2020 08:27
@jdneo jdneo requested a review from akaroml April 16, 2020 08:27
@jdneo
Copy link
Member Author

jdneo commented Apr 16, 2020

@Eskibear @testforstephen @akaroml

Please take a look when you have time. The A/B testing work will be addressed in another PR, to avoid too much code change in one PR.

if (contextManager.getContextValue(Context.MAVEN_ENABLED)) {
projectKinds.push({
label: BuildTool.Maven,
detail: "Use Maven as the build tool of your Java project",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apache Maven is a software project management and comprehension tool. Based on the concept of a project object model (POM), Maven can manage a project's build, reporting and documentation from a central piece of information.

so would something like "Use Maven to manage your project" be simple and concise?

},
);

switch (choice.label) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

choice is possible to be undefined, if you have more than one project kinds.

@jdneo jdneo merged commit 98fd081 into master Apr 20, 2020
@jdneo jdneo deleted the cs/create-project branch April 20, 2020 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants