search

openshift / jenkins-plugin

Apache License 2.0
81 stars 50 forks source link

Modified code to check for whitespace #86

Closed oatmealraisin closed 8 years ago

oatmealraisin commented 8 years ago

Modified setters, contructors, and length checks to account for leading and trailing whitespace.

@gabemontero @bparees ptal?

oatmealraisin commented 8 years ago

@gabemontero I've added the check throughout, good catch, thank you! ptal?

oatmealraisin commented 8 years ago

@gabemontero I've fixed the unnecessary check, ptal?

gabemontero commented 8 years ago

IGTM ... will wait for test job to hit button.

gabemontero commented 8 years ago

no vargrant hiccup this time around ... job should complete soon