Skip to content

Commit d59a2d2

Browse files
committed
ci: fix coveralls integration for github actions
The nbaztec coveralls-jacoco plugin auto-detects GitHub Actions and parses PR/branch info from GitHub's environment variables. Only need to set CI_BRANCH since GitHub_ACTIONS and GITHUB_TOKEN are auto-provided. Assisted-By: Claude Code
1 parent c0a5903 commit d59a2d2

2 files changed

Lines changed: 0 additions & 12 deletions

File tree

.github/workflows/java-ci.yaml

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -467,20 +467,10 @@ jobs:
467467
- name: Push coverage to Coveralls
468468
env:
469469
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
470-
CI_NAME: github
471-
CI_BUILD_NUMBER: ${{ github.run_number }}
472-
CI_BUILD_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
473470
CI_BRANCH: ${{ github.ref_name }}
474-
CI_PULL_REQUEST: ${{ github.event.pull_request.number }}
475471
run: |
476472
if [ -z "$COVERALLS_REPO_TOKEN" ]; then
477473
echo "ERROR: COVERALLS_REPO_TOKEN is not set"
478474
exit 1
479475
fi
480476
./gradlew coverallsJacoco
481-
echo "=== Coveralls request payload ==="
482-
if [ -f build/coveralls-request.json ]; then
483-
cat build/coveralls-request.json | head -100
484-
else
485-
echo "No coveralls-request.json file found"
486-
fi

build.gradle

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,8 +206,6 @@ coverallsJacoco {
206206
reportSourceSets = publishedProjects.collectMany {
207207
it.sourceSets.main.allSource.srcDirs
208208
}
209-
// Debug: write request payload to file to see what's being sent
210-
coverallsRequest = file("build/coveralls-request.json")
211209
}
212210

213211
def javaFormatSources = providers.provider {

0 commit comments

Comments
 (0)