Merge pull request #35 from basil/spotbugs

Remove unnecessary SpotBugs exclusion
jenkinsci · Dec 21, 2022 · a706108 · a706108
2 parents 90a00fc + d232c9f
commit a706108
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/workflow/libs/SCMSourceRetriever.java b/src/main/java/org/jenkinsci/plugins/workflow/libs/SCMSourceRetriever.java
@@ -172,7 +172,6 @@ private static <T> T retrySCMOperation(TaskListener listener, Callable<T> task)
         return ret;
     }
 
-    @SuppressFBWarnings(value = "RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE", justification = "apparently bogus complaint about redundant nullcheck in try-with-resources")
     static void doRetrieve(String name, boolean changelog, @NonNull SCM scm, String libraryPath, FilePath target, Run<?, ?> run, TaskListener listener) throws Exception {
         // Adapted from CpsScmFlowDefinition:
         SCMStep delegate = new GenericSCMStep(scm);