Skip to content

fix: replace Codecov upload with GitHub Job Summary for coverage#91

Merged
spawnia merged 5 commits into
masterfrom
remove-codecov
Jul 7, 2026
Merged

fix: replace Codecov upload with GitHub Job Summary for coverage#91
spawnia merged 5 commits into
masterfrom
remove-codecov

Conversation

@spawnia

@spawnia spawnia commented Jul 7, 2026

Copy link
Copy Markdown
Member

Codecov uploads without a token hit a global rate limit (HTTP 429),
causing intermittent CI failures unrelated to the actual test results.
Closes #44

Switch to actions/upload-code-coverage@v1 — GitHub's native code quality
feature. PHPUnit generates a Cobertura XML report, which is uploaded to
GitHub's coverage API. The github-code-quality[bot] then posts per-file
coverage breakdowns directly on each PR.

Remove .codecov.yml — no longer needed.

Codecov uploads without a token hit a global rate limit (HTTP 429),
causing intermittent CI failures unrelated to the actual test results.
Closes #44

Switch to PHPUnit's built-in --coverage-text, piped to $GITHUB_STEP_SUMMARY.
Coverage is now visible directly in the GitHub Actions run without any
third-party service, token, or upload step.

Remove .codecov.yml — no longer needed.
@spawnia
spawnia marked this pull request as ready for review July 7, 2026 11:01
@github-code-quality

github-code-quality Bot commented Jul 7, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: PHP

PHP / code-coverage/phpunit

The overall coverage in the remove-codecov branch is 89%. Coverage data for the master branch is not yet available.

Show a code coverage summary of the most covered files.
File master remove-codecov a846589 +/-
SafeCast.php 100%
GenomicRegion.php 100%
LightcyclerExpo...erXmlParser.php 98%
Microplate/Abst...tMicroplate.php 95%
Microplate/Coordinates.php 94%
InterOp/RunParameters.php 91%
Microplate/Coor...inateSystem.php 90%
Tecan/TecanProtocol.php 89%
InterOp/LaneResult.php 88%
StringUtil.php 63%

Updated July 07, 2026 13:17 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to eliminate intermittent CI failures caused by unauthenticated Codecov uploads hitting rate limits by removing the Codecov integration and switching coverage reporting to a GitHub-native mechanism.

Changes:

  • Update the code-coverage job to generate a coverage XML file and upload it using actions/upload-code-coverage@v1.
  • Add job-level GITHUB_TOKEN permissions for coverage upload.
  • Remove .codecov.yml since Codecov is no longer used.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/validate.yml Replaces Codecov upload step with GitHub’s upload-code-coverage action and adjusts coverage generation.
.codecov.yml Removes Codecov configuration now that Codecov is no longer used.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/validate.yml Outdated
Comment thread .github/workflows/validate.yml
Comment thread .github/workflows/validate.yml Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/validate.yml
@spawnia
spawnia merged commit 5ce81cf into master Jul 7, 2026
30 checks passed
@spawnia
spawnia deleted the remove-codecov branch July 7, 2026 14:49
github-actions Bot pushed a commit that referenced this pull request Jul 7, 2026
## [6.13.2](v6.13.1...v6.13.2) (2026-07-07)

### Bug Fixes

* replace Codecov upload with GitHub Job Summary for coverage ([#91](#91)) ([5ce81cf](5ce81cf))
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 6.13.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

Error uploading to https://codecov.io

3 participants