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

[FIXES JENKINS-15235] Add whitespace trimming of branch name. #115

Merged
merged 2 commits into from Dec 19, 2012

Conversation

Projects
None yet
3 participants
@thesam
Copy link

thesam commented Dec 19, 2012

  • Trim branch name before storing it, since a valid branch name does
    not begin or end with whitespace.
  • Use same code for setting name in constructor and in setName().
    Previously, there were no check for valid values in setName().

thesam added some commits Dec 19, 2012

[FIXES JENKINS-15235]
- Trim branch name before storing it, since a valid branch name does
not begin or end with whitespace.
- Use same code for setting name in constructor and in setName().
Previously, there were no check for valid values in setName().
@buildhive

This comment has been minimized.

Copy link

buildhive commented Dec 19, 2012

Jenkins » git-plugin #127 SUCCESS
This pull request looks good
(what's this?)

ndeloof added a commit that referenced this pull request Dec 19, 2012

Merge pull request #115 from thesam/jenkins15235
[FIXES JENKINS-15235] Add whitespace trimming of branch name.

@ndeloof ndeloof merged commit 442eecd into jenkinsci:master Dec 19, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.