knowing what was poked is useful

kohsuke · kohsuke · commit a094fc1b36dd · 2012-04-09T14:40:39.000-07:00
diff --git a/src/main/java/com/cloudbees/jenkins/GitHubWebHook.java b/src/main/java/com/cloudbees/jenkins/GitHubWebHook.java
@@ -171,9 +171,10 @@ public void processGitHubPayload(String payload, Class<? extends Trigger> trigge
                     GitHubTrigger trigger = (GitHubTrigger) job.getTrigger(triggerClass);
                     if (trigger!=null) {
                         LOGGER.fine("Considering to poke "+job.getFullDisplayName());
-                        if (trigger.getGitHubRepositories().contains(changedRepository))
+                        if (trigger.getGitHubRepositories().contains(changedRepository)) {
+                            LOGGER.info("Poked "+job.getFullDisplayName());
                             trigger.onPost();
-                        else
+                        } else
                             LOGGER.fine("Skipped "+job.getFullDisplayName()+" because it doesn't have a matching repository.");
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -171,9 +171,10 @@ public void processGitHubPayload(String payload, Class<? extends Trigger> trigge`
`171`	`171`	`GitHubTrigger trigger = (GitHubTrigger) job.getTrigger(triggerClass);`
`172`	`172`	`if (trigger!=null) {`
`173`	`173`	`LOGGER.fine("Considering to poke "+job.getFullDisplayName());`
`174`		`- if (trigger.getGitHubRepositories().contains(changedRepository))`
	`174`	`+ if (trigger.getGitHubRepositories().contains(changedRepository)) {`
	`175`	`+ LOGGER.info("Poked "+job.getFullDisplayName());`
`175`	`176`	`trigger.onPost();`
`176`		`- else`
	`177`	`+ } else`
`177`	`178`	`LOGGER.fine("Skipped "+job.getFullDisplayName()+" because it doesn't have a matching repository.");`
`178`	`179`	`}`
`179`	`180`	`}`