Improved logging format.

jglick · jglick · commit 2b2b1b988e2e · 2016-04-28T06:29:02.000-04:00
diff --git a/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java b/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java
@@ -108,7 +108,6 @@ private boolean runPolling() {
 
             public void run() {
                 if (runPolling()) {
-                    String name = " #" + job.getNextBuildNumber();
                     GitHubPushCause cause;
                     try {
                         cause = new GitHubPushCause(getLogFile(), pushBy);
@@ -117,9 +116,9 @@ public void run() {
                         cause = new GitHubPushCause(pushBy);
                     }
                     if (asParameterizedJobMixIn(job).scheduleBuild(cause)) {
-                        LOGGER.info("SCM changes detected in " + job.getName() + ". Triggering " + name);
+                        LOGGER.info("SCM changes detected in " + job.getFullName() + ". Triggering #" + job.getNextBuildNumber());
                     } else {
-                        LOGGER.info("SCM changes detected in " + job.getName() + ". Job is already in the queue");
+                        LOGGER.info("SCM changes detected in " + job.getFullName() + ". Job is already in the queue");
                     }
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,6 @@ private boolean runPolling() {`
`108`	`108`
`109`	`109`	`public void run() {`
`110`	`110`	`if (runPolling()) {`
`111`		`- String name = " #" + job.getNextBuildNumber();`
`112`	`111`	`GitHubPushCause cause;`
`113`	`112`	`try {`
`114`	`113`	`cause = new GitHubPushCause(getLogFile(), pushBy);`
`@@ -117,9 +116,9 @@ public void run() {`
`117`	`116`	`cause = new GitHubPushCause(pushBy);`
`118`	`117`	`}`
`119`	`118`	`if (asParameterizedJobMixIn(job).scheduleBuild(cause)) {`
`120`		`- LOGGER.info("SCM changes detected in " + job.getName() + ". Triggering " + name);`
	`119`	`+ LOGGER.info("SCM changes detected in " + job.getFullName() + ". Triggering #" + job.getNextBuildNumber());`
`121`	`120`	`} else {`
`122`		`- LOGGER.info("SCM changes detected in " + job.getName() + ". Job is already in the queue");`
	`121`	`+ LOGGER.info("SCM changes detected in " + job.getFullName() + ". Job is already in the queue");`
`123`	`122`	`}`
`124`	`123`	`}`
`125`	`124`	`}`