Skip to content

Commit

Permalink
chore(pom): move enforcer plugin to top pom.xml, and pass the compile (
Browse files Browse the repository at this point in the history
…#32)

Signed-off-by: daniel-y <[email protected]>
Co-authored-by: SSpirits <[email protected]>
  • Loading branch information
daniel-y and ShadowySpirits committed Sep 27, 2023
1 parent 2abd65a commit 04a720c
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 31 deletions.
18 changes: 0 additions & 18 deletions controller/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -131,24 +131,6 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<version>1.4.1</version>
<executions>
<execution>
<id>enforce</id>
<goals>
<goal>enforce</goal>
</goals>
<configuration>
<rules>
<requireUpperBoundDeps/>
</rules>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>build-helper-maven-plugin</artifactId>
Expand Down
49 changes: 36 additions & 13 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@
<slf4j.version>2.0.9</slf4j.version>
<logback.version>1.4.11</logback.version>
<etcd.version>0.7.6</etcd.version>
<awaitility.version>4.2.0</awaitility.version>

<!-- Testing related -->
<mockito-core.version>5.5.0</mockito-core.version>
Expand Down Expand Up @@ -98,13 +97,17 @@
</exclusions>
</dependency>

<!-- Below artifact are not directly used by the project, but are required by the dependencies of the project. -->
<dependency>
<groupId>org.awaitility</groupId>
<artifactId>awaitility</artifactId>
<version>${awaitility.version}</version>
<scope>test</scope>
<groupId>com.google.protobuf</groupId>
<artifactId>protobuf-java-util</artifactId>
<version>3.24.0</version>
</dependency>
<dependency>
<groupId>com.squareup.okio</groupId>
<artifactId>okio-jvm</artifactId>
<version>3.2.0</version>
</dependency>

</dependencies>
</dependencyManagement>

Expand Down Expand Up @@ -137,13 +140,6 @@
<artifactId>assertj-core</artifactId>
<version>${assertj-core.version}</version>
<scope>test</scope>
<exclusions>
<exclusion>
<!-- Exclude the byte-buddy, since the mockito-core already has a higher version of it. -->
<groupId>net.bytebuddy</groupId>
<artifactId>byte-buddy</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>org.mockito</groupId>
Expand All @@ -163,6 +159,14 @@
<version>${awaitility.version}</version>
<scope>test</scope>
</dependency>

<!-- Below dependencies are not directly used by the project, but are required by the dependencies of the project. -->
<dependency>
<groupId>net.bytebuddy</groupId>
<artifactId>byte-buddy</artifactId>
<version>1.14.6</version>
<scope>test</scope>
</dependency>
</dependencies>

<build>
Expand All @@ -188,6 +192,25 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<version>1.4.1</version>
<executions>
<execution>
<id>enforce</id>
<goals>
<goal>enforce</goal>
</goals>
<configuration>
<rules>
<requireUpperBoundDeps/>
<banDuplicatePomDependencyVersions/>
</rules>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</build>
</project>

0 comments on commit 04a720c

Please sign in to comment.