You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2020/09/19 20:01:35 UTC

[logging-pipelines] branch master updated: Enable spotbugs in CI

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

mattsicker pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-pipelines.git


The following commit(s) were added to refs/heads/master by this push:
     new cb1deba  Enable spotbugs in CI
cb1deba is described below

commit cb1debaf5008991de487dc8a74f85f948aa1d113
Author: Matt Sicker <bo...@gmail.com>
AuthorDate: Sat Sep 19 15:00:39 2020 -0500

    Enable spotbugs in CI
---
 log4j/Jenkinsfile | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/log4j/Jenkinsfile b/log4j/Jenkinsfile
index 9bcafcd..8937f2c 100644
--- a/log4j/Jenkinsfile
+++ b/log4j/Jenkinsfile
@@ -58,15 +58,14 @@ pipeline {
             parallel {
                 stage('Ubuntu') {
                     steps {
-                        mvn '-Dmaven.test.failure.ignore=true verify'
+                        mvn '-Dmaven.test.failure.ignore=true verify spotbugs:spotbugs'
                         junit '**/target/*-reports/TEST-*.xml'
-                        // TODO: would be nice to be able to include checkstyle, cpd, pmd, and spotbugs,
-                        //       but current site build takes too long
+                        // TODO: would be nice to be able to include checkstyle, cpd, and pmd
                         recordIssues sourceCodeEncoding: 'UTF-8',
                                 tools: [
                                     mavenConsole(),
                                     errorProne(),
-                                    java(),
+                                    spotBugs(),
                                     taskScanner(highTags: 'FIXME', normalTags: 'TODO', includePattern: '**/*.java', excludePattern: '*/target/**')
                                 ]
                     }