More careful checkstyle regex for todos (#237)

The current regex erroneously catches // TODO(dtrump): Make america() return GREAT again.
palantir · Mar 16, 2018 · d5cddee · d5cddee
1 parent 5d90aba
commit d5cddee
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/.baseline/checkstyle/checkstyle.xml b/.baseline/checkstyle/checkstyle.xml
@@ -328,7 +328,7 @@
             <property name="message" value="Redundant ''static'' modifier."/>
         </module>
         <module name="RegexpSinglelineJava">
-            <property name="format" value="(\/\/TODO)|(\/\/ TODO:)|(\/\/ TODO\(\))|(\/\/ TODO\(.*\):[^ ])|(\/\/ TODO\(.*\)[^:])"/>
+            <property name="format" value="(\/\/TODO)|(\/\/ TODO:)|(\/\/ TODO\(\))|(\/\/ TODO\([a-z]*\):[^ ])|(\/\/ TODO\([a-z]*\)[^:])"/>
             <property name="message" value="TODO format: // TODO(flastname): explanation"/>
         </module>
         <module name="RegexpSinglelineJava">