Skip to content

Commit

Permalink
Update enforcer to allow running tests on JDK 21
Browse files Browse the repository at this point in the history
Also removes redundant enforcer settings

Signed-off-by: Andrew Pielage <[email protected]>
  • Loading branch information
Pandrex247 committed Oct 6, 2023
1 parent 3846d40 commit ad3a634
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 28 deletions.
2 changes: 1 addition & 1 deletion core/core-parent/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@
<configuration>
<rules>
<requireJavaVersion>
<version>[${jdk.version},21)</version>
<version>[${jdk.version},)</version>
<message>You need at least JDK11</message>
</requireJavaVersion>
<requireMavenVersion>
Expand Down
2 changes: 1 addition & 1 deletion core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@

<!-- build versions -->
<jdk.version>11</jdk.version>
<maven.enforcer.plugin.version>3.1.0</maven.enforcer.plugin.version>
<maven.enforcer.plugin.version>3.4.1</maven.enforcer.plugin.version>
<command-security-plugin.version>1.0.13</command-security-plugin.version>
<command.security.maven.plugin.isFailureFatal>false</command.security.maven.plugin.isFailureFatal>
<maven.jar.plugin.version>3.3.0</maven.jar.plugin.version>
Expand Down
26 changes: 0 additions & 26 deletions nucleus/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -405,32 +405,6 @@
</pluginManagement>

<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<version>${maven.enforcer.plugin.version}</version>
<executions>
<execution>
<id>enforce-versions</id>
<goals>
<goal>enforce</goal>
</goals>
<configuration>
<rules>
<requireJavaVersion>
<version>[${jdk.version},18.9)</version>
<message>You need at least JDK11</message>
</requireJavaVersion>
<requireMavenVersion>
<version>[3.0.3,3.2.1],[3.2.3,)</version>
<message>You need Maven greater than 3.0.3 (3.2.2 not supported)</message>
</requireMavenVersion>
</rules>
</configuration>
</execution>
</executions>
</plugin>

<plugin>
<groupId>org.glassfish.build</groupId>
<artifactId>glassfishbuild-maven-plugin</artifactId>
Expand Down

0 comments on commit ad3a634

Please sign in to comment.