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
EXEC-105: fix documentation #112
EXEC-105: fix documentation #112
Conversation
@@ -169,7 +169,7 @@ executor.execute(cmdLine, resultHandler); | |||
|
|||
// some time later the result handler callback was invoked so we | |||
// can safely request the exit value | |||
int exitValue = resultHandler.waitFor(); | |||
resultHandler.waitFor(); |
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 would also show the exitValue = resultHandler.getExitValue(); call as the comment suggests.
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.
Hi @ecki,
Does the PR look ok to you now?
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.
Looks good - maybe in the future we should use snippets or at least have a copy in a test class.
you might need to add the changes.xml entry?
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.
@ecki updated changes.xml
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.
In general, don't update changes.xml, if every PR did that, they would all create merge problems. Note that "dev" is for Apache IDs and "due-to" is for contributor IDs that can be tied so something, like a GitHub account.
Replace "dev" with "due-to" public GitHub name. Sentences end in a period.
EXEC-105: fixed documentation