olamy commented on a change in pull request #3:
URL: https://github.com/apache/maven-jenkins-lib/pull/3#discussion_r815376523
##########
File path: vars/asfMavenTlpPlgnBuild.groovy
##########
@@ -213,6 +224,15 @@ def doCreateTask( os, jdk, maven, tasks, first, plan,
taskContext )
}
}
}
+ if(recordReporting) {
+ recordIssues id: "${os}-jdk${jdk}", name: "Static Analysis",
aggregatingResults: true, enabledForFailure: true, tools: [mavenConsole(),
java(), checkStyle(), spotBugs(), pmdParser(), errorProne()]
+ jacoco inclusionPattern: '**/org/apache/maven/**/*.class',
+ exclusionPattern: '',
+ execPattern: '**/target/jacoco.exec',
Review comment:
good point to include IT test in coverage.
I have setup this to be in the same file with
https://github.com/apache/maven-compiler-plugin/commit/7e83e0d0604dc2d04f21dfea788b849c28e71a54
and covered lines increased from build #16 here
https://ci-maven.apache.org/job/Maven/job/maven-compiler-plugin-test-olamy/job/ci-reporting/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
