Skip to content

Commit ef9289a

Browse files
committed
GHA: Make deploy.sh compatible with aurora
When building artifacts for the shading check, build them in a single Maven command to avoid having to install the parent POM. This approach works both with master (which flattens POMs) and aurora (which does not). Change-Id: I4da093b38ec1a750125c694ec2e8891c0b1547bb Reviewed-on: https://review.couchbase.org/c/couchbase-jvm-clients/+/228492 Tested-by: David Nault <[email protected]> Reviewed-by: David Nault <[email protected]>
1 parent 2e13e6f commit ef9289a

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

.github/scripts/deploy.sh

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,8 @@ set -x
1717
./mvnw --batch-mode --file core-io-deps/pom.xml clean install
1818

1919
# Improper shading should have been caught during PR verification, but let's double check.
20-
./mvnw --batch-mode --file test-utils/pom.xml clean install
21-
./mvnw --batch-mode --file core-io/pom.xml install -Dmaven.test.skip=true -Dmaven.javadoc.skip=true
20+
./mvnw --batch-mode clean install --projects test-utils,core-io,java-client,tracing-opentelemetry -Dmaven.test.skip=true -Dmaven.javadoc.skip=true
2221
cd core-io ; ./shade-check.sh ; cd ..
23-
./mvnw --batch-mode --file java-client/pom.xml install -Dmaven.test.skip=true -Dmaven.javadoc.skip=true
24-
./mvnw --batch-mode --file tracing-opentelemetry/pom.xml install -Dmaven.test.skip=true -Dmaven.javadoc.skip=true
2522
cd tracing-opentelemetry ; ./shade-check.sh ; cd ..
2623

2724
./mvnw --batch-mode --settings deploy-settings.xml deploy -Dgpg.signer=bc -Dsurefire.rerunFailingTestsCount=1 --activate-profiles ${MAVEN_PROFILE}

0 commit comments

Comments
 (0)