Add option to not send the "hello!" message to non-whitelisted

pull request submissions

This is helpful where you have multiple Jenkins jobs, all looking
at the same repository - rather than spamming the user with multiple
"Hello!" messages, you can pick one job to spam the user loudly, whilst
letting all your other jobs silently obey the build/whitelist/etc comments

src/main/java/org/jenkinsci/plugins/ghprb/Ghprb.java

@@ -182,6 +182,10 @@ public boolean isTriggerPhrase(String comment) {
     public boolean ifOnlyTriggerPhrase() {
         return trigger.getOnlyTriggerPhrase();
     }
+
+    public boolean suppressTestingRequest() {
+        return trigger.getSuppressTestingRequest();
+    }
 
     public boolean isWhitelisted(GHUser user) {
         return trigger.getPermitAll()

src/main/java/org/jenkinsci/plugins/ghprb/GhprbPullRequest.java

@@ -79,7 +79,7 @@ public class GhprbPullRequest {
         if (helper.isWhitelisted(author)) {
             accepted = true;
             shouldRun = true;
-        } else {
+        } else if (!helper.suppressTestingRequest()) {
             logger.log(Level.INFO, "Author of #{0} {1} on {2} not in whitelist!", new Object[] { id, author.getLogin(), reponame });
             repo.addComment(id, GhprbTrigger.getDscp().getRequestForTestingPhrase());
         }

src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java

@@ -60,6 +60,7 @@ public class GhprbTrigger extends GhprbTriggerBackwardsCompatible {
     private final String triggerPhrase;
     private final String buildDescTemplate;
     private final Boolean onlyTriggerPhrase;
+    private final Boolean suppressTestingRequest;
     private final Boolean useGitHubHooks;
     private final Boolean permitAll;
     private String whitelist;
@@ -104,6 +105,7 @@ public GhprbTrigger(String adminlist,
             String triggerPhrase,
             Boolean onlyTriggerPhrase, 
             Boolean useGitHubHooks,
+            Boolean suppressTestingRequest,
             Boolean permitAll,
             Boolean autoCloseFailedPullRequests,
             Boolean displayBuildErrorsOnDownstreamBuilds,
@@ -124,6 +126,7 @@ public GhprbTrigger(String adminlist,
         this.cron = cron;
         this.triggerPhrase = triggerPhrase;
         this.onlyTriggerPhrase = onlyTriggerPhrase;
+        this.suppressTestingRequest = suppressTestingRequest;
         this.useGitHubHooks = useGitHubHooks;
         this.permitAll = permitAll;
         this.autoCloseFailedPullRequests = autoCloseFailedPullRequests;
@@ -401,6 +404,10 @@ public String getTriggerPhrase() {
     public Boolean getOnlyTriggerPhrase() {
         return onlyTriggerPhrase != null && onlyTriggerPhrase;
     }
+
+    public Boolean getSuppressTestingRequest() {
+        return suppressTestingRequest;
+    }
 
     public Boolean getUseGitHubHooks() {
         return useGitHubHooks != null && useGitHubHooks;

src/main/resources/org/jenkinsci/plugins/ghprb/GhprbTrigger/config.groovy

@@ -18,6 +18,9 @@ f.advanced() {
   f.entry(field: "onlyTriggerPhrase", title: "Only use trigger phrase for build triggering") {
     f.checkbox() 
   }
+  f.entry(field: "suppressTestingRequest", title: "Don't send request for approval message for non-whitelisted builders") {
+    f.checkbox() 
+  }
   f.entry(field: "autoCloseFailedPullRequests", title: _("Close failed pull request automatically?")) {
     f.checkbox(default: descriptor.autoCloseFailedPullRequests) 
   }

src/main/resources/org/jenkinsci/plugins/ghprb/GhprbTrigger/config.jelly.tmp

@@ -21,6 +21,9 @@
 	<f:entry title="Only use trigger phrase for build triggering" field="onlyTriggerPhrase">
 	  <f:checkbox />
 	</f:entry>
+	<f:entry title="Don't send request for approval message for non-whitelisted builders" field="suppressTestingRequest">
+	  <f:checkbox />
+	</f:entry>
 	<f:entry title="${%Close failed pull request automatically?}" field="autoCloseFailedPullRequests">
 	  <f:checkbox default="${descriptor.autoCloseFailedPullRequests}"/>
 	</f:entry>