From 0f857065bd8afe7f58bc29202b4a27c5f724b591 Mon Sep 17 00:00:00 2001
From: Bukkit/Spigot <noreply+git-bukkit@papermc.io>
Date: Sat, 19 Feb 2022 09:33:06 +1100
Subject: [PATCH] #730: Added a checkstyle rule to check for missing package
 declarations.

Added a suppression to ignore the test sources.

By: blablubbabc <lukas@wirsindwir.de>
---
 paper-api/checkstyle-suppressions.xml | 9 +++++++++
 paper-api/checkstyle.xml              | 3 +++
 paper-api/pom.xml                     | 1 +
 3 files changed, 13 insertions(+)
 create mode 100644 paper-api/checkstyle-suppressions.xml

diff --git a/paper-api/checkstyle-suppressions.xml b/paper-api/checkstyle-suppressions.xml
new file mode 100644
index 000000000..94366ef66
--- /dev/null
+++ b/paper-api/checkstyle-suppressions.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0"?>
+<!DOCTYPE suppressions PUBLIC
+          "-//Checkstyle//DTD SuppressionFilter Configuration 1.2//EN"
+          "https://checkstyle.org/dtds/suppressions_1_2.dtd">
+
+<suppressions>
+    <!-- Don't check for missing package declarations in tests-->
+    <suppress checks="JavadocPackage" files=".*[\\/]src[\\/]test[\\/].*\.java$"/>
+</suppressions>
diff --git a/paper-api/checkstyle.xml b/paper-api/checkstyle.xml
index 8b6241649..c658f38f1 100644
--- a/paper-api/checkstyle.xml
+++ b/paper-api/checkstyle.xml
@@ -20,6 +20,9 @@
         <property name="message" value="Line has trailing spaces."/>
     </module>
 
+    <!-- See https://checkstyle.sourceforge.io/config_javadoc.html#JavadocPackage-->
+    <module name="JavadocPackage"/>
+
     <module name="TreeWalker">
         <!-- See https://checkstyle.org/config_javadoc.html -->
         <module name="AtclauseOrder"/>
diff --git a/paper-api/pom.xml b/paper-api/pom.xml
index b8c801951..298e00fee 100644
--- a/paper-api/pom.xml
+++ b/paper-api/pom.xml
@@ -217,6 +217,7 @@
                         </executions>
                         <configuration>
                             <configLocation>checkstyle.xml</configLocation>
+                            <suppressionsLocation>checkstyle-suppressions.xml</suppressionsLocation>
                             <includeTestSourceDirectory>true</includeTestSourceDirectory>
                         </configuration>
                         <dependencies>