Skip to content

Remove phpdbg from coverage target#44

Merged
f3l1x merged 1 commit intocontributte:masterfrom
ohmyfelix:chore/php-coverage
Mar 16, 2026
Merged

Remove phpdbg from coverage target#44
f3l1x merged 1 commit intocontributte:masterfrom
ohmyfelix:chore/php-coverage

Conversation

@ohmyfelix
Copy link
Contributor

Summary

Remove phpdbg from the coverage target and run coverage through php instead.

Motivation

This repository is part of the org-wide migration tracked in contributte/contributte#73.

Changes

  • replace -p phpdbg with -p php in both coverage target variants in Makefile
  • keep the existing coverage output paths unchanged

Testing

  • make tests
  • make coverage locally (Xdebug or PCOV extension or PHPDBG SAPI is not available in this environment)
  • CI passes

Replace phpdbg with php in the coverage target to match the org-wide coverage migration tracked in contributte/contributte#73.
@codecov
Copy link

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

@f3l1x f3l1x merged commit 87445c2 into contributte:master Mar 16, 2026
10 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