Checkstyle Fail build if checkstyle fails

.travis.yml

@@ -34,6 +34,8 @@ script:
   - ./gradlew test
   - ./gradlew codeCoverageReport
   - ./gradlew docs > /dev/null # build the javadoc
+  - ./gradlew checkstyleMain
+  - ./gradlew checkstyleTest
 
 after_success:
   - bash <(curl -s https://codecov.io/bash)

alchemy/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

build.gradle

@@ -17,6 +17,19 @@ javadoc {
   source = 'src/main/java'
 }
 
+checkstyle {
+  ignoreFailures = false
+}
+
+checkstyleMain {
+  ignoreFailures = false
+}
+
+checkstyleTest {
+  ignoreFailures = false
+}
+
+
 task docs(type: Javadoc) {
   destinationDir = file("$buildDir/docs/all")
 }
@@ -45,8 +58,12 @@ allprojects {
 
 subprojects {
 
+  checkstyleMain {
+    ignoreFailures = false
+  }
+
   checkstyleTest {
-    ignoreFailures = true
+    ignoreFailures = false
   }
 
   afterEvaluate {

conversation/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

core/build.gradle

@@ -49,13 +49,10 @@ signArchives {
     }
 }
 
-checkstyleTest {
-    ignoreFailures = true
-}
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
   task testJar(type: Jar) {

core/src/main/java/com/ibm/watson/developer_cloud/service/WatsonService.java

@@ -134,7 +134,7 @@ protected OkHttpClient configureHttpClient() {
         .build();
 
     builder.connectionSpecs(Arrays.asList(spec, ConnectionSpec.CLEARTEXT));
-    
+
     return builder.build();
   }
 

core/src/test/java/com/ibm/watson/developer_cloud/util/WaitFor.java

@@ -21,7 +21,9 @@
 
 public class WaitFor {
 
-  private WaitFor() {}
+  private WaitFor() {
+
+  }
 
   /**
    * Static method used to wait for a specific condition to be satisfied.

dialog/build.gradle

@@ -49,11 +49,9 @@ signArchives {
     }
 }
 
-checkstyleTest {
-    ignoreFailures = true
-}
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 task testJar(type: Jar) {

discovery/build.gradle

@@ -50,12 +50,12 @@ signArchives {
 }
 
 checkstyleTest {
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 dependencies {

discovery/src/test/java/com/ibm/watson/developer_cloud/discovery/v1/DiscoveryServiceTest.java

@@ -73,7 +73,14 @@
 
 import static org.junit.Assert.assertEquals;
 
-import java.io.*;
+import java.io.FileReader;
+import java.io.FileNotFoundException;
+import java.io.InputStream;
+import java.io.ByteArrayInputStream;
+import java.io.File;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.io.BufferedWriter;
 import java.util.HashSet;
 import java.util.Set;
 

document-conversion/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

java-sdk/build.gradle

@@ -58,12 +58,12 @@ signArchives {
 }
 
 checkstyleTest {
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 task testJar(type: Jar) {

language-translation/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

language-translator/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

natural-language-classifier/build.gradle

@@ -50,12 +50,12 @@ signArchives {
 }
 
 checkstyleTest {
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 dependencies {

natural-language-understanding/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

personality-insights/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

retrieve-and-rank/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

speech-to-text/build.gradle

@@ -49,8 +49,13 @@ signArchives {
     }
 }
 
+checkstyleTest {
+    ignoreFailures = false
+}
+
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

text-to-speech/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

tone-analyzer/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

tradeoff-analytics/build.gradle

@@ -51,6 +51,7 @@ signArchives {
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
+    ignoreFailures = false
 }
 
 dependencies {

visual-recognition/build.gradle

@@ -50,12 +50,12 @@ signArchives {
 }
 
 checkstyleTest {
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 checkstyle {
     configFile = rootProject.file('checkstyle.xml')
-    ignoreFailures = true
+    ignoreFailures = false
 }
 
 dependencies {