Skip to content

Commit f3aa366

Browse files
committed
Optimize CI: remove redundant Gradle tasks and fix Windows fail-fast
- Remove redundant './gradlew assemble' on Linux and macOS. The 'build' task already includes 'assemble' as a dependency, so running it separately just duplicates work. - Remove './gradlew publishToMavenLocal' on all three OSes. No downstream job consumes the published artifacts. - Add 'fail-fast: false' to Windows matrix for consistency with Linux and macOS, preventing one Java version failure from cancelling the other. Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>
1 parent 7ad2810 commit f3aa366

File tree

1 file changed

+3
-13
lines changed

1 file changed

+3
-13
lines changed

.github/workflows/test_build_multi_platform.yml

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ jobs:
2828
Build-ad-windows:
2929
needs: spotless
3030
strategy:
31+
fail-fast: false
3132
matrix:
3233
java: [21, 25]
3334
name: Build and Test Anomaly Detection Plugin on Windows
@@ -46,9 +47,6 @@ jobs:
4647
- name: Build and Run Tests
4748
run: |
4849
./gradlew build -x spotlessJava
49-
- name: Publish to Maven Local
50-
run: |
51-
./gradlew publishToMavenLocal
5250
- name: Multi Nodes Integration Testing
5351
run: |
5452
./gradlew integTest -PnumNodes=3
@@ -83,12 +81,10 @@ jobs:
8381
- name: Checkout AD
8482
uses: actions/checkout@v6
8583

86-
- name: Assemble / build / mavenlocal / integTest
84+
- name: Build and Run Tests
8785
run: |
8886
chown -R 1000:1000 `pwd`
89-
su `id -un 1000` -c "./gradlew assemble &&
90-
./gradlew build -x spotlessJava &&
91-
./gradlew publishToMavenLocal &&
87+
su `id -un 1000` -c "./gradlew build -x spotlessJava &&
9288
./gradlew integTest -PnumNodes=3"
9389
9490
Build-ad-macos:
@@ -113,9 +109,6 @@ jobs:
113109
- name: Checkout AD
114110
uses: actions/checkout@v6
115111

116-
- name: Assemble anomaly-detection
117-
run: |
118-
./gradlew assemble
119112
- name: Build and Run Tests
120113
run: |
121114
./gradlew build -x spotlessJava
@@ -168,9 +161,6 @@ jobs:
168161
files: ./build/reports/jacoco/integTest/jacocoIntegTestReport.xml
169162
disable_search: true
170163
flags: plugin_integtest
171-
- name: Publish to Maven Local
172-
run: |
173-
./gradlew publishToMavenLocal
174164
- name: Multi Nodes Integration Testing
175165
run: |
176166
./gradlew integTest -PnumNodes=3

0 commit comments

Comments
 (0)