From 494ecdd79f552f6200a6f341c36d9a5e1886df5c Mon Sep 17 00:00:00 2001
From: David Fuelling <sappenin@gmail.com>
Date: Wed, 5 Feb 2020 14:43:27 -0700
Subject: [PATCH] Update Checkstyle to 8.29 (#434)

Signed-off-by: David Fuelling <sappenin@gmail.com>
---
 checkstyle.xml | 17 +++++++++++------
 pom.xml        |  2 +-
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/checkstyle.xml b/checkstyle.xml
index f6a40d72..6851a85e 100644
--- a/checkstyle.xml
+++ b/checkstyle.xml
@@ -53,6 +53,11 @@
         <property name="eachLine" value="true"/>
     </module>
 
+    <module name="LineLength">
+        <property name="max" value="120"/>
+        <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
+    </module>
+
     <module name="TreeWalker">
         <module name="OuterTypeFilename"/>
         <module name="IllegalTokenText">
@@ -66,10 +71,6 @@
             <property name="allowByTailComment" value="true"/>
             <property name="allowNonPrintableEscapes" value="true"/>
         </module>
-        <module name="LineLength">
-            <property name="max" value="120"/>
-            <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
-        </module>
         <module name="AvoidStarImport"/>
         <module name="OneTopLevelClass"/>
         <module name="NoLineWrap"/>
@@ -209,11 +210,15 @@
         <module name="JavadocMethod">
             <property name="scope" value="public"/>
             <property name="allowMissingParamTags" value="true"/>
-            <property name="allowMissingThrowsTags" value="true"/>
             <property name="allowMissingReturnTag" value="true"/>
+            <property name="allowedAnnotations" value="Override, Test"/>
+            <property name="validateThrows" value="false"/>
+        </module>
+        <module name="MissingJavadocMethodCheck">
+            <property name="scope" value="public"/>
             <property name="minLineCount" value="2"/>
+            <property name="allowMissingPropertyJavadoc" value="true"/>
             <property name="allowedAnnotations" value="Override, Test"/>
-            <property name="allowThrowsTagsForSubclasses" value="true"/>
         </module>
         <module name="MethodName">
             <property name="format" value="^[a-z][a-z0-9][a-zA-Z0-9_]*$"/>
diff --git a/pom.xml b/pom.xml
index 627a46ae..5a641204 100644
--- a/pom.xml
+++ b/pom.xml
@@ -424,7 +424,7 @@
             <dependency>
               <groupId>com.puppycrawl.tools</groupId>
               <artifactId>checkstyle</artifactId>
-              <version>8.23</version>
+              <version>8.29</version>
             </dependency>
           </dependencies>
           <executions>