Skip to content

Migrate coverage reporting from Coveralls to Codecov#9

Merged
BenMorel merged 2 commits into
masterfrom
copilot/migrate-to-codecov
Feb 27, 2026
Merged

Migrate coverage reporting from Coveralls to Codecov#9
BenMorel merged 2 commits into
masterfrom
copilot/migrate-to-codecov

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 27, 2026

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

Changes

  • .github/workflows/ci.yml: Simplifies the coverage step (drops the erroneous mkdir -p mkdir -p build/logs, outputs clover.xml directly), replaces Coveralls upload with codecov/codecov-action@v5 using CODECOV_TOKEN.
  • README.md: Swaps Coveralls badge for Codecov badge.
  • composer.json: Removes php-coveralls/php-coveralls from require-dev.
Original prompt

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

Changes required

1. .github/workflows/ci.yml

In the phpunit job:

  • In the "Run PHPUnit with coverage" step, change the clover output path from build/logs/clover.xml to clover.xml (and remove the mkdir -p build/logs line if present).

  • Replace the "Upload coverage report to Coveralls" step 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 }}

2. README.md

Replace the Coveralls badge with:

[![Coverage](https://codecov.io/github/brick/date-time-doctrine/graph/badge.svg)](https://codecov.io/github/brick/date-time-doctrine)

3. composer.json

Remove php-coveralls/php-coveralls from require-dev if present.

This pull request was created from Copilot chat.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>
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
Copy Markdown

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 ☂️

@BenMorel BenMorel merged commit f5d38f0 into master Feb 27, 2026
16 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