Merge pull request #25 from jenkinsci/remove-old-api

Remove the old "fromRun" method which does not take listener as parameter
jenkinsci · Sep 3, 2020 · 8abdf81 · 8abdf81
2 parents cebddb9 + b0802fd
commit 8abdf81
Showing 1 changed file with 0 additions and 13 deletions.
diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksPublisherFactory.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksPublisherFactory.java
@@ -4,7 +4,6 @@
 import java.util.Optional;
 
 import edu.hm.hafner.util.VisibleForTesting;
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import hudson.model.Job;
 import hudson.model.Run;
 import hudson.model.TaskListener;
@@ -60,18 +59,6 @@ protected Optional<ChecksPublisher> createPublisher(final Job<?, ?> job, final T
         return Optional.empty();
     }
 
-    /**
-     * Returns a suitable publisher for the run.
-     *
-     * @param run
-     *         a Jenkins run
-     * @return a publisher suitable for the job
-     */
-    @SuppressFBWarnings(value = "NP_NONNULL_PARAM_VIOLATION", justification = "restore to 0.1.0 version api")
-    public static ChecksPublisher fromRun(final Run<?, ?> run) {
-        return fromRun(run, null, new JenkinsFacade());
-    }
-
     /**
      * Returns a suitable publisher for the run.
      *