You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by sh...@apache.org on 2023/03/15 13:43:21 UTC

[bookkeeper] branch master updated: Disable code coverage and codecov report (#3863)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 8aee66a2f7 Disable code coverage and codecov report (#3863)
8aee66a2f7 is described below

commit 8aee66a2f767f346265be7391d887254923e1ab4
Author: Lishen Yao <ya...@gmail.com>
AuthorDate: Wed Mar 15 21:43:13 2023 +0800

    Disable code coverage and codecov report (#3863)
    
    ### Motivation
    
    There're two reasons that we want to disable the code coverage.
    
    1. The current report result is not accurate.
    2. We can't get the PR's unit test's code coverage because of the apache Codecov permission.
---
 .github/workflows/bk-ci.yml | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/.github/workflows/bk-ci.yml b/.github/workflows/bk-ci.yml
index 1b4fe9a8c2..e8d7782114 100644
--- a/.github/workflows/bk-ci.yml
+++ b/.github/workflows/bk-ci.yml
@@ -178,13 +178,7 @@ jobs:
           if [[ ! -z "${{ matrix.module }}" ]]; then
             projects_list="-pl ${{ matrix.module }}"
           fi
-          mvn -Pcode-coverage -B -nsu $projects_list verify ${{ matrix.test_args }}
-
-      - name: Upload coverage to Codecov
-        if: ${{ matrix.flag }} != 'shell'
-        uses: codecov/codecov-action@v3
-        with:
-          flags: ${{ matrix.flag }}
+          mvn -B -nsu $projects_list verify ${{ matrix.test_args }}
 
       - name: Aggregates all test reports to ./test-reports and ./surefire-reports directories
         if: ${{ always() }}