Closed soasme closed 9 years ago
@nemccarthy code review please
It's being tested and just works :)
This is a good change but would it make more sense as a global config setting? And maybe move skip there too?
+1 for some global config but there also seems a need for this to be configured per project. Thanks for the PR!
I think
test this please
is too verbose to type (and is easy to make typo). Allowing customizeciBuildPhrases
is much more flexible, just likeciSkipPhrases
.PS. I have no idea how to test jenkins plugin, so this pr is still not tested :(