This is an automated email from the ASF dual-hosted git repository.

panjuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new e5989ca3cd4 Refactor : refactor the ci workflow with corresponding JDK 
version (#24635)
e5989ca3cd4 is described below

commit e5989ca3cd42d661dc11f2f4c2416689787530ca
Author: 孙念君 Nianjun Sun <[email protected]>
AuthorDate: Thu Mar 16 11:02:52 2023 +0800

    Refactor : refactor the ci workflow with corresponding JDK version (#24635)
---
 .github/workflows/ci.yml         | 23 +++++++----------------
 .github/workflows/nightly-ci.yml |  2 +-
 2 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index a44219895ce..ae426b01026 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -56,7 +56,7 @@ env:
 
 jobs:
   ci:
-    name: CI - JDK ${{ matrix.java-version }} on ${{ matrix.os }}
+    name: CI - on JDK ${{ matrix.java-version }}
     runs-on: ubuntu-latest
     timeout-minutes: 60
     strategy:
@@ -78,18 +78,16 @@ jobs:
         run: ./mvnw -T1C -B -ntp clean install
       - name: Build examples with Maven
         run: ./mvnw -T1C -B -f examples/pom.xml clean package -DskipTests
-      - name: Setup JDK 8 for Test
-        uses: actions/setup-java@v3
-        with:
-          distribution: 'temurin'
-          java-version: 8
-      - name: Run tests with JDK 8
+      - name: Run tests with JDK ${{ matrix.java-version }}
         run: ./mvnw -T1C -B -ntp -fae test
 
   test-coverage:
     if: github.repository == 'apache/shardingsphere'
-    name: Test coverage report
+    name: Test coverage report on JDK ${{ matrix.java-version }}
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        java-version: [ 11, 19 ]
     steps:
       - uses: actions/checkout@v3
       - name: Cache Maven Repos
@@ -102,15 +100,8 @@ jobs:
       - uses: actions/setup-java@v3
         with:
           distribution: 'temurin'
-          java-version: 11
+          java-version: ${{ matrix.java-version }}
       - name: Build with Maven
         run: ./mvnw -T1C -B -ntp clean install -DskipTests
-      - name: Setup JDK 8 for Test
-        uses: actions/setup-java@v3
-        with:
-          distribution: 'temurin'
-          java-version: 8
-      - name: Test with Maven
-        run: ./mvnw -T1C -B -ntp test cobertura:cobertura -Djacoco.skip=false
       - name: Upload to Codecov
         run: bash <(curl -s https://codecov.io/bash)
diff --git a/.github/workflows/nightly-ci.yml b/.github/workflows/nightly-ci.yml
index 4e5251416db..d76499c066d 100644
--- a/.github/workflows/nightly-ci.yml
+++ b/.github/workflows/nightly-ci.yml
@@ -37,7 +37,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, macos-latest, windows-latest ]
-        java-version: [ 11, 17, 19 ]
+        java-version: [ 8, 11, 17, 19 ]
     steps:
       - name: Support long paths in Windows
         if: matrix.os == 'windows-latest'

Reply via email to