[helix] branch master updated: Ensure the CI result report action is done even test fails.

[ji...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new d0167e4  Ensure the CI result report action is done even test fails.
d0167e4 is described below

commit d0167e4abc261b3f814af817e1fdf5cadcaede38
Author: Jiajun Wang <jj...@linkedin.com>
AuthorDate: Thu Apr 22 13:52:09 2021 -0700

    Ensure the CI result report action is done even test fails.
---
 .github/workflows/Helix-CI.yml    | 3 ++-
 .github/workflows/Helix-PR-CI.yml | 3 +++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/Helix-CI.yml b/.github/workflows/Helix-CI.yml
index 28d50f4..745b783 100644
--- a/.github/workflows/Helix-CI.yml
+++ b/.github/workflows/Helix-CI.yml
@@ -22,6 +22,7 @@ jobs:
       run: mvn -q -fae test
     - name: Print Tests Results
       run: .github/scripts/printTestResult.sh
+      if: ${{ success() || failure() }}
     - name: Upload to Codecov
-      if: ${{ github.event_name == 'push' }}
       run: bash <(curl -s https://codecov.io/bash)
+      if: ${{ github.event_name == 'push' && (success() || failure()) }}
diff --git a/.github/workflows/Helix-PR-CI.yml b/.github/workflows/Helix-PR-CI.yml
index 3249d43..b2d31df 100644
--- a/.github/workflows/Helix-PR-CI.yml
+++ b/.github/workflows/Helix-PR-CI.yml
@@ -18,3 +18,6 @@ jobs:
       run: mvn clean install -Dmaven.test.skip.exec=true
     - name: Run All Tests
       run: mvn -q -fae test
+    - name: Print Tests Results
+      run: .github/scripts/printTestResult.sh
+      if: ${{ success() || failure() }}