Add checkstyle errors for non-inclusive terms (opensearch-project#1782)

Signed-off-by: Peter Nied <petern@amazon.com>
ochprince · Apr 21, 2022 · f3aa8be · f3aa8be
1 parent 9dc2d5e
commit f3aa8be
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 5 deletions.
diff --git a/build.gradle b/build.gradle
@@ -372,11 +372,9 @@ checkstyle {
 }
 
 tasks.withType(Checkstyle) {
+    showViolations true
     reports {
-        showViolations false
-        ignoreFailures = true
-        xml.required = true
-        html.required = true
+        ignoreFailures = false
     }
 }
 

diff --git a/config/checkstyle/sun_checks.xml b/config/checkstyle/sun_checks.xml
@@ -27,7 +27,7 @@
       https://checkstyle.org/config.html#Checker
       <property name="basedir" value="${basedir}"/>
   -->
-  <property name="severity" value="error"/>
+  <property name="severity" value="ignore"/>
 
   <property name="fileExtensions" value="java, properties, xml"/>
 
@@ -183,4 +183,18 @@
 
   </module>
 
+  <module name="RegexpSingleline">
+    <property name="format" value="whitelist"/>
+    <property name="ignoreCase" value="true"/>
+    <property name="message" value="Usage should be switched to an allow* based pattern" />
+    <property name="severity" value="warning"/>
+  </module>
+
+  <module name="RegexpSingleline">
+    <property name="format" value="master"/>
+    <property name="ignoreCase" value="true"/>
+    <property name="message" value="Usage should be switched to cluster manager" />
+    <property name="severity" value="warning"/>
+  </module>
+
 </module>