Skip to content

Replace phpdbg with php in coverage target#26

Open
ohmyfelix wants to merge 1 commit intocontributte:masterfrom
ohmyfelix:chore/php-coverage-target
Open

Replace phpdbg with php in coverage target#26
ohmyfelix wants to merge 1 commit intocontributte:masterfrom
ohmyfelix:chore/php-coverage-target

Conversation

@ohmyfelix
Copy link
Contributor

Summary

  • replace -p phpdbg with -p php in the coverage target for both CI and local runs

Motivation

This keeps contributte/messenger aligned with the org-wide coverage migration tracked in contributte/contributte#73.

Changes

  • update the CI coverage command in Makefile
  • update the local coverage command in Makefile

Testing

  • make coverage locally (php coverage is blocked here because this environment has no Xdebug or PCOV extension installed)
  • make tests locally (fails in this environment on PHP 8.5.3 because DI/MessengerExtension.handler.phpt errors with "Cannot use object of type Generator as array" and DI/MessengerExtension.transport.phpt requires php-redis 4.3.0 or higher)
  • CI passes

@codecov
Copy link

codecov bot commented Mar 11, 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 ☂️

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.

1 participant