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
[FIX JENKINS-44108][JENKINS-44111] Remoting Work Directory Support #95
Conversation
|
||
/** | ||
* @param workingDirectory The Working directory set on the configuration of the node. | ||
* @return <p>the remoting parameter to set the workDir, |
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'm pretty sure this return tag will include all the lines without needing the <p>
wrapper. Otherwise, perhaps it'd be better to put the docs above the tags?
*/ | ||
@NonNull | ||
@Restricted(NoExternalUse.class) | ||
public String getWorkDirParam(@Nonnull String workingDirectory){ |
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.
You're using two different @Nonnull
annotations in the same signature!
@@ -0,0 +1,5 @@ | |||
<div> | |||
In <i>Remoting work directory</i> is an internal data storage, which may be used by Remoting to store caches, logs and other metadata. |
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.
Sentence could make more sense by changing "In" to "The" at the beginning.
JENKINS-44108
JENKINS-44111