Skip to content

Remove phpdbg from coverage target#30

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

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

Conversation

@ohmyfelix
Copy link
Contributor

Summary

Motivation

  • phpdbg is being removed from coverage targets across Contributte so coverage runs use plain php

Changes

  • update both CI and local coverage commands in Makefile

Testing

  • make coverage starts the updated php command
  • Coverage completes locally (php in this environment has no Xdebug/PCOV, and PHPDBG is intentionally no longer used)
  • CI passes

@codecov
Copy link

codecov bot commented Mar 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.92%. Comparing base (b90e7fd) to head (bcaf85a).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   78.04%   76.92%   -1.13%     
==========================================
  Files           2        3       +1     
  Lines         123      117       -6     
==========================================
- Hits           96       90       -6     
  Misses         27       27              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@f3l1x f3l1x marked this pull request as ready for review March 16, 2026 19:12
@f3l1x f3l1x merged commit 8119299 into contributte:master Mar 16, 2026
8 of 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