Skip to content
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

CLI and Launcher documentation #1160

Merged
merged 10 commits into from Sep 29, 2015
Merged

Conversation

cescoffier
Copy link
Contributor

Provide the CLI and Launcher documentation

Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
…mmands.

Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
@cescoffier
Copy link
Contributor Author

@vietj you asked for it... please review.

code as verticles and use the Vert.x `Starter` class as your main class. This is the same main class used when running
Vert.x at the command line and therefore allows you to specify command line arguments, such as `-instances` in order
to scale your application more easily.
code as verticles and use the Vert.x `Launcher` class as your main class. This is the same main class used when
Copy link
Member

Choose a reason for hiding this comment

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

recommend : use {@link io...Lauchner} so user can go to Javadoc

Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>
@cescoffier
Copy link
Contributor Author

@vietj - I've updated the code to fix your comment. Can you merge ?

@vietj
Copy link
Member

vietj commented Sep 29, 2015

yes

@vietj vietj merged commit 113fc70 into eclipse-vertx:master Sep 29, 2015
@vietj vietj removed the to review label Sep 29, 2015
@cescoffier cescoffier deleted the cli-launcher-doc branch July 20, 2016 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants