Merge pull request #30 from jenkinsci/remove-beta-restriction

Remove beta restrictions
jenkinsci · Sep 25, 2020 · 4456c01 · 4456c01
2 parents 536774b + e360644
commit 4456c01
Show file tree

Hide file tree

Showing 9 changed files with 3 additions and 35 deletions.
diff --git a/pom.xml b/pom.xml
@@ -101,14 +101,13 @@
         <groupId>org.revapi</groupId>
         <artifactId>revapi-maven-plugin</artifactId>
         <configuration>
-          <versionFormat>[0-9.]*</versionFormat> 
-          <checkDependencies>false</checkDependencies>
           <analysisConfiguration>
             <revapi.ignore combine.children="append">
               <item>
                 <regex>true</regex>
-                <code>java.missing.*</code>
-                <justification>Dependencies are not being checked, so they are reported as missing</justification>
+                <code>java.annotation.removed</code>
+                <annotationType>org.kohsuke.accmod.Restricted</annotationType>
+                <justification>Beta restriction should be safe to remove.</justification>
               </item>
             </revapi.ignore>
           </analysisConfiguration>

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksAction.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksAction.java
@@ -5,13 +5,9 @@
 import edu.umd.cs.findbugs.annotations.CheckForNull;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 /**
  * An action of a check. It can be used to create actions like re-run or automatic formatting.
  */
-@Restricted(Beta.class)
 public class ChecksAction {
     private final String label;
     private final String description;

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksAnnotation.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksAnnotation.java
@@ -2,15 +2,11 @@
 
 import java.util.Optional;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 import static java.util.Objects.*;
 
 /**
  * An annotation for specific lines of code.
  */
-@Restricted(Beta.class)
 @SuppressWarnings("PMD.DataClass")
 public class ChecksAnnotation {
     private final String path;

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksConclusion.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksConclusion.java
@@ -1,14 +1,10 @@
 package io.jenkins.plugins.checks.api;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 /**
  * Conclusion for a specific check. One of the conclusions (except the {@link ChecksConclusion#NONE}, which is used to
  * represent "no conclusion yet") should be provided when the {@link ChecksStatus} of a check is set to
  * {@link ChecksStatus#COMPLETED}.
  */
-@Restricted(Beta.class)
 public enum ChecksConclusion {
     NONE, ACTION_REQUIRED, SKIPPED, CANCELED, TIME_OUT, FAILURE, NEUTRAL, SUCCESS
 }
diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksDetails.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksDetails.java
@@ -6,15 +6,11 @@
 import java.util.List;
 import java.util.Optional;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 import static java.util.Objects.requireNonNull;
 
 /**
  * Details of a check. This class is a top class which contains all parameters needed for a check.
  */
-@Restricted(Beta.class)
 @SuppressWarnings("PMD.DataClass")
 public class ChecksDetails {
     private final String name;

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksImage.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksImage.java
@@ -5,13 +5,9 @@
 import edu.umd.cs.findbugs.annotations.CheckForNull;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 /**
  * An image of a check. Users may use a image to show the code coverage, issues trend, etc.
  */
-@Restricted(Beta.class)
 @SuppressWarnings("PMD.DataClass")
 public class ChecksImage {
     private final String alt;

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksOutput.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksOutput.java
@@ -5,16 +5,12 @@
 import java.util.List;
 import java.util.Optional;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 import static java.util.Objects.*;
 
 /**
  * An output of a check. The output usually contains the most useful information like summary, description,
  * annotations, etc.
  */
-@Restricted(Beta.class)
 public class ChecksOutput {
     private final String title;
     private final String summary;

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksPublisher.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksPublisher.java
@@ -3,13 +3,10 @@
 import hudson.model.Job;
 import hudson.model.TaskListener;
 import io.jenkins.plugins.util.PluginLogger;
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
 
 /**
  * A general publisher for publishing checks to different platforms.
  */
-@Restricted(Beta.class)
 public abstract class ChecksPublisher {
     /**
      * Publishes checks to platforms.

diff --git a/src/main/java/io/jenkins/plugins/checks/api/ChecksStatus.java b/src/main/java/io/jenkins/plugins/checks/api/ChecksStatus.java
@@ -1,12 +1,8 @@
 package io.jenkins.plugins.checks.api;
 
-import org.kohsuke.accmod.Restricted;
-import org.kohsuke.accmod.restrictions.Beta;
-
 /**
  * Status for a specific check.
  */
-@Restricted(Beta.class)
 public enum ChecksStatus {
     NONE, QUEUED, IN_PROGRESS, COMPLETED
 }