mirror of https://github.com/citusdata/citus.git
Remove Code Climate coverage upload steps from GitHub Actions workflow (#8182)
DESCRIPTION: Remove Code Climate coverage upload steps from GitHub Actions workflow CI: remove Code Climate coverage reporting (cc-test-reporter) and related jobs; keep Codecov as source of truth * **Why** Code Climate’s test-reporter has been archived; their download/API path is no longer served, which breaks our CC upload step (`cc-test-reporter …` ends up downloading HTML/404). * **What changed** * Drop the Code Climate formatting/artifact steps from the composite action `.github/actions/upload_coverage/action.yml`. * Delete the `upload-coverage` job that aggregated and pushed to Code Climate (`cc-test-reporter sum-coverage` / `upload-coverage`). * **Impact** * Codecov uploads remain; coverage stays visible via Codecov. * No test/build behavior change—only removes a failing reporter path.pull/8341/head
parent
6ef177c1a3
commit
1ce314337e
|
|
@ -13,15 +13,3 @@ runs:
|
||||||
token: ${{ inputs.codecov_token }}
|
token: ${{ inputs.codecov_token }}
|
||||||
verbose: true
|
verbose: true
|
||||||
gcov: true
|
gcov: true
|
||||||
- name: Create codeclimate coverage
|
|
||||||
run: |-
|
|
||||||
lcov --directory . --capture --output-file lcov.info
|
|
||||||
lcov --remove lcov.info -o lcov.info '/usr/*'
|
|
||||||
sed "s=^SF:$PWD/=SF:=g" -i lcov.info # relative pats are required by codeclimate
|
|
||||||
mkdir -p /tmp/codeclimate
|
|
||||||
cc-test-reporter format-coverage -t lcov -o /tmp/codeclimate/${{ inputs.flags }}.json lcov.info
|
|
||||||
shell: bash
|
|
||||||
- uses: actions/upload-artifact@v4.6.0
|
|
||||||
with:
|
|
||||||
path: "/tmp/codeclimate/*.json"
|
|
||||||
name: codeclimate-${{ inputs.flags }}
|
|
||||||
|
|
|
||||||
|
|
@ -404,30 +404,6 @@ jobs:
|
||||||
with:
|
with:
|
||||||
flags: ${{ env.PG_MAJOR }}_citus_upgrade
|
flags: ${{ env.PG_MAJOR }}_citus_upgrade
|
||||||
codecov_token: ${{ secrets.CODECOV_TOKEN }}
|
codecov_token: ${{ secrets.CODECOV_TOKEN }}
|
||||||
upload-coverage:
|
|
||||||
# secret below is not available for forks so disabling upload action for them
|
|
||||||
if: ${{ github.event.pull_request.head.repo.full_name == github.repository || github.event_name != 'pull_request' }}
|
|
||||||
env:
|
|
||||||
CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
container:
|
|
||||||
image: ${{ needs.params.outputs.test_image_name }}:${{ fromJson(needs.params.outputs.pg17_version).full }}${{ needs.params.outputs.image_suffix }}
|
|
||||||
needs:
|
|
||||||
- params
|
|
||||||
- test-citus
|
|
||||||
- test-arbitrary-configs
|
|
||||||
- test-citus-upgrade
|
|
||||||
- test-pg-upgrade
|
|
||||||
steps:
|
|
||||||
- uses: actions/download-artifact@v4.1.8
|
|
||||||
with:
|
|
||||||
pattern: codeclimate*
|
|
||||||
path: codeclimate
|
|
||||||
merge-multiple: true
|
|
||||||
- name: Upload coverage results to Code Climate
|
|
||||||
run: |-
|
|
||||||
cc-test-reporter sum-coverage codeclimate/*.json -o total.json
|
|
||||||
cc-test-reporter upload-coverage -i total.json
|
|
||||||
ch_benchmark:
|
ch_benchmark:
|
||||||
name: CH Benchmark
|
name: CH Benchmark
|
||||||
if: startsWith(github.ref, 'refs/heads/ch_benchmark/')
|
if: startsWith(github.ref, 'refs/heads/ch_benchmark/')
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue