From 538483c1925bace4d8fd078ec6b95300036087fc Mon Sep 17 00:00:00 2001
From: Yi Hsuen <tayyihsuen@gmail.com>
Date: Fri, 29 Apr 2022 21:57:12 +0800
Subject: [PATCH] Upgrade Checkstyle to 10.2

The latest version of Checkstyle is 10.2. At the same time, the scope
property in JavadocMethod was replaced with accessModifiers in
Checkstyle 8.42.

Let's upgrade Checkstyle and change scope to accessModifiers.
---
 build.gradle                     | 2 +-
 config/checkstyle/checkstyle.xml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/build.gradle b/build.gradle
index 46bdf94cb38..108397716bd 100644
--- a/build.gradle
+++ b/build.gradle
@@ -17,7 +17,7 @@ repositories {
 }
 
 checkstyle {
-    toolVersion = '8.29'
+    toolVersion = '10.2'
 }
 
 test {
diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml
index 4c001417aea..8444b3590cf 100644
--- a/config/checkstyle/checkstyle.xml
+++ b/config/checkstyle/checkstyle.xml
@@ -384,7 +384,7 @@
     <!-- Checks that every public method (excluding getters, setters and constructors) has a header comment. -->
     <module name="JavadocMethod">
       <property name="allowedAnnotations" value="Override, Test, BeforeAll, BeforeEach, AfterAll, AfterEach, Subscribe"/>
-      <property name="scope" value="public"/>
+      <property name="accessModifiers" value="public"/>
       <property name="validateThrows" value="false"/>
       <property name="allowMissingParamTags" value="true"/>
       <property name="allowMissingReturnTag" value="true"/>