Skip to content

Commit d2d4899

Browse files
committed
chore(ci): update Maven commands to use -Dmaven.test.skip=true for consistency
1 parent 1894d2b commit d2d4899

4 files changed

Lines changed: 8 additions & 9 deletions

File tree

.github/copilot-instructions.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ Always reference these instructions first and fallback to search or bash command
1515
### Building the Project
1616
- **NEVER CANCEL builds - they take time but will complete successfully**
1717
- **CRITICAL**: Set timeout to 20+ minutes for builds, 60+ minutes for tests
18-
- Clean compile: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw clean compile -B --no-transfer-progress -DskipTests` -- takes ~10.5 minutes
19-
- Full package: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw package -B --no-transfer-progress -DskipTests` -- takes ~4 minutes
20-
- Install to local repository: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw install -B --no-transfer-progress -DskipTests` -- takes ~1.5 minutes
18+
- Clean compile: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw clean compile -B --no-transfer-progress -Dmaven.test.skip=true` -- takes ~10.5 minutes
19+
- Full package: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw package -B --no-transfer-progress -Dmaven.test.skip=true` -- takes ~4 minutes
20+
- Install to local repository: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH" && ./mvnw install -B --no-transfer-progress -Dmaven.test.skip=true` -- takes ~1.5 minutes
2121

2222
### Testing
2323
- **NEVER CANCEL test runs - set 60+ minute timeouts**
@@ -101,7 +101,7 @@ Always reference these instructions first and fallback to search or bash command
101101
## Development Workflow
102102
1. **Setup**: Configure Node.js 22.18.0 in PATH: `export PATH="/tmp/node-v22.18.0-linux-x64/bin:$PATH"`
103103
2. **Initial Build**: Run `./mvnw clean compile` to verify everything builds (~10.5 minutes - be patient)
104-
3. **Install Dependencies**: Run `./mvnw install -DskipTests` for sample app dependencies (~1.5 minutes)
104+
3. **Install Dependencies**: Run `./mvnw install -Dmaven.test.skip=true` for sample app dependencies (~1.5 minutes)
105105
4. **Make Changes**: Edit code in appropriate modules
106106
5. **Format Code**: Run `./mvnw spring-javaformat:apply` and UI `npm run format:fix`
107107
6. **Test Build**: Run build commands to verify changes don't break anything
@@ -146,4 +146,4 @@ spring-boot-admin/ # Root project (19 modules)
146146
├── spring-boot-admin-starter-*/ # Spring Boot auto-configuration
147147
├── mvnw # Maven wrapper
148148
└── pom.xml # Root Maven configuration
149-
```
149+
```

.github/workflows/build-main.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ jobs:
3636

3737
- name: Build with Maven (parallel modules)
3838
run: ./mvnw -B -T 1C --no-transfer-progress install -P coverage
39-
# -T 1C = 1 thread per CPU core for parallel module builds
4039

4140
- name: Upload surefire reports
4241
if: always()
@@ -56,7 +55,7 @@ jobs:
5655
uses: codecov/codecov-action@v6
5756

5857
- name: Publish SNAPSHOT version to GitHub Packages
59-
run: ./mvnw -B -T 1C deploy --no-transfer-progress -DskipTests
58+
run: ./mvnw -B -T 1C deploy --no-transfer-progress -Dmaven.test.skip=true
6059
env:
6160
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
6261

.github/workflows/deploy-documentation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
run: ./mvnw versions:set "-DnewVersion=${{ github.event.inputs.releaseversion }}" --no-transfer-progress
4949

5050
- name: Build with Maven (parallel modules)
51-
run: ./mvnw -B -T 1C --no-transfer-progress package -DskipTests
51+
run: ./mvnw -B -T 1C --no-transfer-progress package -Dmaven.test.skip=true
5252

5353
- name: Build documentation with Maven
5454
run: ./mvnw -B --no-transfer-progress -pl spring-boot-admin-docs site

.github/workflows/release-to-maven-central.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ jobs:
5454
run: ./mvnw versions:set "-DnewVersion=${{ github.event.inputs.releaseversion }}" --no-transfer-progress
5555

5656
- name: Publish package (parallel modules)
57-
run: ./mvnw -B -T 1C deploy --no-transfer-progress -P central-deploy -DskipTests
57+
run: ./mvnw -B -T 1C deploy --no-transfer-progress -P central-deploy -Dmaven.test.skip=true
5858
env:
5959
#TODO: This is a workaround for NEXUS-27902 which uses XStream that fails on JVM >16
6060
JDK_JAVA_OPTIONS: "--add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang.reflect=ALL-UNNAMED --add-opens java.base/java.text=ALL-UNNAMED --add-opens java.desktop/java.awt.font=ALL-UNNAMED"

0 commit comments

Comments
 (0)