Skip to content

Commit

Permalink
Disable code coverage and codecov report (apache#3863)
Browse files Browse the repository at this point in the history
### 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.
  • Loading branch information
yaalsn authored and Anup Ghatage committed Jul 12, 2024
1 parent 189a0d5 commit 2d3848a
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions .github/workflows/bk-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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() }}
Expand Down

0 comments on commit 2d3848a

Please sign in to comment.