Make extension optional so it will be easier to delete the API later

jglick · jglick · commit bbeaebf3d95d · 2025-03-04T16:55:29.000-05:00
diff --git a/src/main/java/org/jenkinsci/plugins/docker/workflow/declarative/AbstractDockerAgent.java b/src/main/java/org/jenkinsci/plugins/docker/workflow/declarative/AbstractDockerAgent.java
@@ -142,7 +142,7 @@ public boolean isAllowed(String groovyResourceUrl) {
         }
     }
 
-    @Extension public static final class Compat implements CompatibilityLoader {
+    @Extension(optional = true) public static final class Compat implements CompatibilityLoader {
         private static final Set<String> CLASSES = Set.of(
             "org.jenkinsci.plugins.docker.workflow.declarative.AbstractDockerPipelineScript",
             "org.jenkinsci.plugins.docker.workflow.declarative.DockerPipelineScript",

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public boolean isAllowed(String groovyResourceUrl) {`
`142`	`142`	`}`
`143`	`143`	`}`
`144`	`144`
`145`		`- @Extension public static final class Compat implements CompatibilityLoader {`
	`145`	`+ @Extension(optional = true) public static final class Compat implements CompatibilityLoader {`
`146`	`146`	`private static final Set<String> CLASSES = Set.of(`
`147`	`147`	`"org.jenkinsci.plugins.docker.workflow.declarative.AbstractDockerPipelineScript",`
`148`	`148`	`"org.jenkinsci.plugins.docker.workflow.declarative.DockerPipelineScript",`