Skip to content

Commit 2be59fe

Browse files
committed
Speed up builds (#1647)
When testing SQL Client templates, we need to build dependencies first (at this moment, Pg and MySQL clients). With this change, the ci.yml workflow action tests the target module in two steps instead of one: - build the target module skipping tests - run tests Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
1 parent 71be788 commit 2be59fe

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/ci-4.x.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ jobs:
8383
profile: 'Oracle-23'
8484
module: 'vertx-oracle-client'
8585
runs-on: ${{ matrix.os }}
86+
env:
87+
PROFILE_ARG: ${{ matrix.profile && format('-P {0}', matrix.profile) || '' }}
8688
steps:
8789
- name: Checkout
8890
uses: actions/checkout@v2
@@ -91,10 +93,12 @@ jobs:
9193
with:
9294
java-version: ${{ matrix.jdk }}
9395
distribution: temurin
94-
- name: Run tests
96+
- name: Build without tests
97+
run: |
98+
mvn -s .github/maven-ci-settings.xml -q clean install -B $PROFILE_ARG -pl :${{ matrix.module }} -am -DskipTests
99+
- name: Run tests for the target module
95100
run: |
96-
PROFILE_ARG="${{ matrix.profile && format('-P {0}', matrix.profile) || '' }}"
97-
mvn -s .github/maven-ci-settings.xml -q clean verify -B $PROFILE_ARG -pl :${{ matrix.module }} -am
101+
mvn -s .github/maven-ci-settings.xml -q verify -B $PROFILE_ARG -pl :${{ matrix.module }}
98102
Deploy:
99103
name: Deploy to OSSRH
100104
if: ${{ github.repository_owner == 'eclipse-vertx' && (github.event_name == 'push' || github.event_name == 'schedule') }}

0 commit comments

Comments
 (0)