Skip to content

Commit 72f891a

Browse files
markpollackWillam2004
authored andcommitted
Fix main-push-fast.yml to test only affected modules
Remove -amd flag from Maven commands which was causing tests for ALL modules that depend on the changed module. For spring-ai-commons, this meant testing nearly the entire project instead of just the commons module. - Phase 2: Remove -amd from verify command to test only specific modules - Deploy: Remove -amd from deploy command for consistency - Phase 1 still uses -am to build required dependencies This ensures the fast workflow truly tests only what changed. Signed-off-by: Mark Pollack <[email protected]> Signed-off-by: 家娃 <[email protected]>
1 parent ba95cc6 commit 72f891a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/main-push-fast.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ jobs:
142142
echo "INFO: Phase 1 - Building dependencies (this may take a few minutes)..."
143143
./mvnw -B -q -T 1C -DskipTests -pl "$MODS" -am install
144144
echo "INFO: Phase 2 - Running tests for affected modules..."
145-
./mvnw -B -q -T 1C -Pci-fast-integration-tests -DfailIfNoTests=false -pl "$MODS" -amd verify
145+
./mvnw -B -q -T 1C -Pci-fast-integration-tests -DfailIfNoTests=false -pl "$MODS" verify
146146
echo "INFO: Testing complete"
147147
fi
148148
@@ -155,4 +155,4 @@ jobs:
155155
MODS="${{ steps.mods.outputs.modules }}"
156156
echo "INFO: Deploying affected modules to Artifactory: $MODS"
157157
# Skip tests during deploy since we already ran them
158-
./mvnw -B -q -s settings.xml -DskipTests -pl "$MODS" -amd deploy
158+
./mvnw -B -q -s settings.xml -DskipTests -pl "$MODS" deploy

0 commit comments

Comments
 (0)