Disallow class-level warning suppression

`@SuppressWarnings` should be applied to narrowest scope possible.
airlift · May 29, 2024 · 3ea2b5c · 3ea2b5c
1 parent 1253ecf
commit 3ea2b5c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,5 +1,6 @@
 Airbase 157
 * Checkstyle updates:
+ - Disallow class-level warning suppression
  - Require empty line before record definition
 
 Airbase 156

diff --git a/airbase-policy/src/main/resources/checkstyle/airbase-checks.xml b/airbase-policy/src/main/resources/checkstyle/airbase-checks.xml
@@ -105,6 +105,10 @@
     <module name="TreeWalker">
         <module name="SuppressWarningsHolder" />
 
+        <module name="SuppressWarnings">
+            <property name="tokens" value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, ANNOTATION_DEF, RECORD_DEF" />
+        </module>
+
         <module name="SuppressionXpathSingleFilter">
             <property name="checks" value="RedundantModifier"/>
             <property name="query" value="//(CLASS_DEF|RECORD_DEF)/OBJBLOCK/*/MODIFIERS/*"/>