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

@@ -139,6 +139,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

@@ -80,7 +80,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

@@ -55,6 +55,7 @@ public class GhprbTrigger extends GhprbTriggerBackwardsCompatible {
     private final String cron;
     private final String triggerPhrase;
     private final Boolean onlyTriggerPhrase;
+    private final Boolean suppressTestingRequest;
     private final Boolean useGitHubHooks;
     private final Boolean permitAll;
     private String whitelist;
@@ -87,6 +88,7 @@ public GhprbTrigger(String adminlist,
             String cron, 
             String triggerPhrase, 
             Boolean onlyTriggerPhrase, 
+            Boolean suppressTestingRequest,
             Boolean useGitHubHooks, 
             Boolean permitAll,
             Boolean autoCloseFailedPullRequests, 
@@ -106,6 +108,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;
@@ -321,6 +324,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.jelly

@@ -12,6 +12,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>