This is an automated email from the ASF dual-hosted git repository.

mxm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new dcd9ff0  add post build action to record errorpron/checkstyle/findbugs 
on precommit java
     new 1b4d904  Merge pull request #7666: Add post build action to record 
errorprone/checkstyle/findbugs on precommit java
dcd9ff0 is described below

commit dcd9ff01011f0ff128e8a0e31f157c98876e4a04
Author: Michael Luckey <[email protected]>
AuthorDate: Wed Jan 30 00:46:15 2019 +0100

    add post build action to record errorpron/checkstyle/findbugs on precommit 
java
---
 .test-infra/jenkins/job_PreCommit_Java.groovy               | 13 +++++++++++++
 .../groovy/org/apache/beam/gradle/BeamModulePlugin.groovy   |  4 ++--
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/.test-infra/jenkins/job_PreCommit_Java.groovy 
b/.test-infra/jenkins/job_PreCommit_Java.groovy
index 31086d3..6ed262e 100644
--- a/.test-infra/jenkins/job_PreCommit_Java.groovy
+++ b/.test-infra/jenkins/job_PreCommit_Java.groovy
@@ -34,5 +34,18 @@ PrecommitJobBuilder builder = new PrecommitJobBuilder(
 builder.build {
   publishers {
     archiveJunit('**/build/test-results/**/*.xml')
+    recordIssues {
+      tools {
+        errorProne()
+        java()
+        checkStyle {
+          pattern('**/build/reports/checkstyle/*.xml')
+        }
+        findBugs {
+          pattern('**/build/reports/findbugs/*.xml')
+        }
+      }
+      enabledForFailure(true)
+    }
   }
 }
diff --git 
a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy 
b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
index efcb2f9..3573acc 100644
--- a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
+++ b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
@@ -761,8 +761,8 @@ class BeamModulePlugin implements Plugin<Project> {
         }
         project.tasks.withType(FindBugs) {
           reports {
-            html.enabled = true
-            xml.enabled = false
+            html.enabled = false
+            xml.enabled = true
           }
         }
       }

Reply via email to