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

Update RemoteBuildConfiguration.java #46

Merged
merged 3 commits into from Aug 9, 2018

Conversation

wincrasher
Copy link

When using triggerRemoteJob from pipeline, if we enter multiple parameters, by default, the pipeline script editor add multiple tabulation to indent the line correctly. When processing them we need to remove those unnecessary tabulation.

When using triggerRemoteJob from pipeline, if we enter multiple parameters, by default, the pipeline script editor add multiple tabulation to indent the line correctly. When processing them we need to remove those unnecessary tabulation.
Copy link

@cashlalala cashlalala left a comment

Choose a reason for hiding this comment

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

awesome, thx

@cashlalala
Copy link

@wincrasher please help to fix the build error, so I can merge it

Forgot to import static org.apache.commons.lang.StringUtils.stripAll;
Change position of strip to match Array instead of List.
@wincrasher
Copy link
Author

@cashlalala Sorry for the multiple commit. Dont have compiler installed so cannot test prior to send the code change.

Copy link

@cashlalala cashlalala left a comment

Choose a reason for hiding this comment

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

okay

@cashlalala cashlalala merged commit a049e10 into jenkinsci:master Aug 9, 2018
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