Skip to content

Conversation

cescoffier
Copy link
Member

Because the project generated by the user contains everything required, no need for 3 quickstarts anymore.

Also, this PR make the instructions more consistent.

@vorburger
Copy link
Contributor

/LGTM

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

@cescoffier could you remove the microprofile-config.properties? Other than that, it looks good.

Your PR is against master. Is it what you wanted to do?

@@ -0,0 +1,2 @@
# Configuration file
key = value
Copy link
Member

Choose a reason for hiding this comment

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

This file should be removed.

Copy link
Member Author

Choose a reason for hiding this comment

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

File deleted.

@cescoffier
Copy link
Member Author

Yes, this PR is against master on purpose. It's an improvement of the guide that should be publish soon (without a release)

@cescoffier cescoffier requested a review from gsmet March 11, 2019 15:14
@gsmet gsmet merged commit b6bf55a into quarkusio:master Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants