Skip to content

Commit

Permalink
Merge pull request #245 from basil/spotbugs
Browse files Browse the repository at this point in the history
Remove unnecessary SpotBugs exclusion
  • Loading branch information
jglick authored Feb 15, 2023
2 parents 0a792d9 + 069aea2 commit bed5ff0
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
package org.jenkinsci.plugins.workflow.steps;

import edu.umd.cs.findbugs.annotations.NonNull;
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import hudson.Extension;
import hudson.FilePath;
import hudson.Util;
Expand Down Expand Up @@ -107,7 +106,6 @@ public static final class Execution extends SynchronousNonBlockingStepExecution<
this.step = step;
}

@SuppressFBWarnings(value = "RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE", justification = "https://github.com/spotbugs/spotbugs/issues/756")
@Override protected Void run() throws Exception {
FilePath file = getContext().get(FilePath.class).child(step.file);
if (ReadFileStep.BASE64_ENCODING.equals(step.encoding)) {
Expand Down

0 comments on commit bed5ff0

Please sign in to comment.