From 1e81c19543c103db5e6ef5022e78d523e50a9333 Mon Sep 17 00:00:00 2001
From: Basil Crow <me@basilcrow.com>
Date: Wed, 2 Nov 2022 14:12:22 -0700
Subject: [PATCH] Remove duplicate SpotBugs defaults

---
 pom.xml | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/pom.xml b/pom.xml
index b7de3ef2b8..da6538a481 100644
--- a/pom.xml
+++ b/pom.xml
@@ -75,13 +75,6 @@
     <hpi-plugin.version>3.35</hpi-plugin.version>
     <stapler-plugin.version>1.20</stapler-plugin.version>
 
-    <!-- Defines a SpotBugs threshold. Use "Low" to discover low-priority bugs.
-         Hint: SpotBugs considers some real NPE risks in Jenkins as low-priority issues, it is recommended to enable it in plugins.
-      -->
-    <spotbugs.threshold>Medium</spotbugs.threshold>
-    <!-- Defines a SpotBugs effort. Use "Max" to maximize the scan depth -->
-    <spotbugs.effort>Default</spotbugs.effort>
-
     <!-- Whether to skip tests during release phase (they are executed in the prepare phase). -->
     <release.skipTests>true</release.skipTests>
     <!-- By default we do not create *-tests.jar. Set to false to allow other plugins to depend on test utilities in yours, using <classifier>tests</classifier>. -->
@@ -684,9 +677,6 @@
       <plugin>
         <groupId>com.github.spotbugs</groupId>
         <artifactId>spotbugs-maven-plugin</artifactId>
-        <configuration>
-          <failOnError>${spotbugs.failOnError}</failOnError>
-        </configuration>
         <executions>
           <execution>
             <id>spotbugs</id>
@@ -699,8 +689,6 @@
               <!--  Instead we configure this in a profile -->
               <xmlOutput>true</xmlOutput>
               <spotbugsXmlOutput>false</spotbugsXmlOutput>
-              <effort>${spotbugs.effort}</effort>
-              <threshold>${spotbugs.threshold}</threshold>
             </configuration>
           </execution>
         </executions>