Skip to content

Commit e7c48a2

Browse files
committed
fix: address review comments
Signed-off-by: neeraj-laad <neeraj.laad@gmail.com>
1 parent 4d7f1a7 commit e7c48a2

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

api/src/main/java/io/strimzi/api/kafka/model/connect/build/MavenArtifactIncludeScope.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@
1010
import java.util.Locale;
1111

1212
/**
13-
* Enum defining the supported Maven dependency include scopes for dependency
14-
* inclusion
13+
* Enum defining the supported Maven dependency include scopes for dependency inclusion
1514
*/
1615
public enum MavenArtifactIncludeScope {
1716
COMPILE,

cluster-operator/src/test/java/io/strimzi/operator/cluster/model/KafkaConnectDockerfileTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
import io.strimzi.api.kafka.model.connect.build.JarArtifactBuilder;
1212
import io.strimzi.api.kafka.model.connect.build.MavenArtifact;
1313
import io.strimzi.api.kafka.model.connect.build.MavenArtifactBuilder;
14+
import io.strimzi.api.kafka.model.connect.build.MavenArtifactIncludeScope;
1415
import io.strimzi.api.kafka.model.connect.build.OtherArtifact;
1516
import io.strimzi.api.kafka.model.connect.build.OtherArtifactBuilder;
1617
import io.strimzi.api.kafka.model.connect.build.PluginBuilder;
@@ -707,7 +708,7 @@ public void testMavenArtifactWithIncludeScope() {
707708
.withGroup("g1")
708709
.withArtifact("a1")
709710
.withVersion("v1")
710-
.withIncludeScope(io.strimzi.api.kafka.model.connect.build.MavenArtifactIncludeScope.RUNTIME)
711+
.withIncludeScope(MavenArtifactIncludeScope.RUNTIME)
711712
.build();
712713

713714
Build connectBuild = new BuildBuilder()
@@ -753,7 +754,7 @@ public void testInsecureMavenArtifactWithIncludeScope() {
753754
.withArtifact("a1")
754755
.withVersion("v1")
755756
.withInsecure(true)
756-
.withIncludeScope(io.strimzi.api.kafka.model.connect.build.MavenArtifactIncludeScope.RUNTIME)
757+
.withIncludeScope(MavenArtifactIncludeScope.RUNTIME)
757758
.build();
758759

759760
Build connectBuild = new BuildBuilder()

0 commit comments

Comments
 (0)