Yuvipanda has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/124825

Change subject: Fix Android Wikipedia app java checkstyles
......................................................................

Fix Android Wikipedia app java checkstyles

- Use checkstyle-aggregate rather than checkstyle. This makes
  it easier to run just checkstyle on both the subprojects
  without having to actually build them
- Set encoding to utf-8 to prevent warning
- Look specifically for checkstyle-result.xml files only to
  prevent errors caused by presence of other checkstyle-* files.

Change-Id: I38ae6767c4cac6a2857b3eae76b61d616b92de11
---
M job-templates.yaml
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/integration/jenkins-job-builder-config 
refs/changes/25/124825/1

diff --git a/job-templates.yaml b/job-templates.yaml
index 022eedd..40b8efb 100644
--- a/job-templates.yaml
+++ b/job-templates.yaml
@@ -264,7 +264,8 @@
       - timestamps
       - ansicolor
     maven:
-      goals: "checkstyle:checkstyle"
-      maven-opts: "-Dcheckstyle.config.location=$WORKSPACE/checkstyle.xml"
+      goals: "checkstyle:checkstyle-aggregate"
+      maven-opts: "-Dcheckstyle.config.location=$WORKSPACE/checkstyle.xml 
-Dencoding=utf-8"
     publishers:
-      - checkstyle-xml
+      - checkstyle-xml:
+          pattern: "**/checkstyle-result.xml"

-- 
To view, visit https://gerrit.wikimedia.org/r/124825
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I38ae6767c4cac6a2857b3eae76b61d616b92de11
Gerrit-PatchSet: 1
Gerrit-Project: integration/jenkins-job-builder-config
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to