Skip to content

Commit

Permalink
Remove the unneeded bom module from the integration-test profile. Add…
Browse files Browse the repository at this point in the history
… the integration-test module to the release profile.

Signed-off-by: James R. Perkins <[email protected]>
  • Loading branch information
jamezp committed Jul 9, 2024
1 parent 78df966 commit cf1e20d
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 4 deletions.
15 changes: 13 additions & 2 deletions integration-tests/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@

<properties>
<maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
<maven.test.skip>false</maven.test.skip>
<skipTests>${maven.test.skip}</skipTests>
<skipItTests>false</skipItTests>
<skipTests>${skipItTests}</skipTests>

<!-- Dependency versions -->
<version.jakarta.ee>10.0.0</version.jakarta.ee>
Expand Down Expand Up @@ -178,6 +178,17 @@
</dependencies>

<build>
<pluginManagement>
<plugins>
<plugin>
<artifactId>maven-release-plugin</artifactId>
<configuration>
<pushChanges>false</pushChanges>
<localCheckout>true</localCheckout>
</configuration>
</plugin>
</plugins>
</pluginManagement>
<plugins>
<plugin>
<artifactId>maven-surefire-plugin</artifactId>
Expand Down
6 changes: 4 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -149,13 +149,14 @@
<profile>
<id>integration-tests</id>
<modules>
<!-- Attempt to ensure the BOM is always built last on Windows -->
<module>bom</module>
<module>integration-tests</module>
</modules>
</profile>
<profile>
<id>release</id>
<modules>
<module>integration-tests</module>
</modules>
<build>
<plugins>
<plugin>
Expand All @@ -167,6 +168,7 @@
<localCheckout>true</localCheckout>
<preparationGoals>clean install</preparationGoals>
<releaseProfiles>gpg-sign,jboss-release</releaseProfiles>
<arguments>-DskipItTests=true</arguments>
<tagNameFormat>@{project.version}</tagNameFormat>
</configuration>
</plugin>
Expand Down

0 comments on commit cf1e20d

Please sign in to comment.