Skip to content

Migrate coverage reporting from Coveralls to Codecov#7

Merged
BenMorel merged 1 commit intomasterfrom
copilot/migrate-to-codecov
Feb 27, 2026
Merged

Migrate coverage reporting from Coveralls to Codecov#7
BenMorel merged 1 commit intomasterfrom
copilot/migrate-to-codecov

Conversation

Copy link
Contributor

Copilot AI commented Feb 27, 2026

Replaces Coveralls with Codecov for coverage reporting, mirroring the migration done in brick/math.

Changes

  • .github/workflows/ci.yml: In all three phpunit jobs (mysql, mariadb, postgres):
    • Fix broken mkdir -p mkdir -p build/logs command; output clover directly to clover.xml
    • Replace php-coveralls upload step with codecov/codecov-action@v5
  • README.md: Swap Coveralls badge for Codecov badge
  • composer.json: Drop php-coveralls/php-coveralls from require-dev
Original prompt

Migrate from Coveralls to Codecov for coverage reporting in brick/geo-doctrine, mirroring the changes made in brick/math#109.

Changes required:

  1. .github/workflows/ci.yml: In the phpunit job, simplify the "Run PHPUnit with coverage" step (remove mkdir -p build/logs, change clover path from build/logs/clover.xml to clover.xml). Replace the "Upload coverage report to Coveralls" step (which runs vendor/bin/php-coveralls with COVERALLS_REPO_TOKEN) with:
- name: Upload coverage to Codecov
  uses: codecov/codecov-action@v5
  env:
    CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
  if: ${{ matrix.php-version == env.COVERAGE_PHP_VERSION }}
  1. README.md: Replace the Coveralls badge with a Codecov badge: [![Coverage](https://codecov.io/github/brick/geo-doctrine/graph/badge.svg)](https://codecov.io/github/brick/geo-doctrine)

  2. composer.json: Remove php-coveralls/php-coveralls from require-dev (if present).

This pull request was created from Copilot chat.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Migrate coverage reporting from Coveralls to Codecov Migrate coverage reporting from Coveralls to Codecov Feb 27, 2026
@BenMorel BenMorel marked this pull request as ready for review February 27, 2026 13:02
@codecov
Copy link

codecov bot commented Feb 27, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>
@BenMorel BenMorel force-pushed the copilot/migrate-to-codecov branch from 641ce93 to c7c9b86 Compare February 27, 2026 15:00
@BenMorel BenMorel merged commit 1d6ddab into master Feb 27, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants