Skip to content

Remove phpdbg from coverage target#544

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

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

Conversation

@ohmyfelix
Copy link
Contributor

Summary

Replace the coverage Makefile target to run Tester with php instead of phpdbg.

Motivation

This aligns the skeleton with the org-wide migration tracked in contributte/contributte#73 and keeps new coverage runs compatible with the shared CI PHP setup.

Changes

  • switch Makefile coverage target from -p phpdbg to -p php

Testing

  • Tests pass locally
  • CI passes
  • Manual verification

Manual verification:

  • make coverage now invokes vendor/bin/tester -s -p php ...
  • local coverage execution is blocked in this environment because PHP lacks Xdebug/PCOV (phpdbg is no longer used)
  • CI uses contributte/.github/.github/actions/setup-php, whose default coverage input is xdebug

@codecov
Copy link

codecov bot commented Mar 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 23.20%. Comparing base (b9919b8) to head (c76b85e).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #544      +/-   ##
==========================================
+ Coverage   18.84%   23.20%   +4.36%     
==========================================
  Files          33       39       +6     
  Lines         276      306      +30     
==========================================
+ Hits           52       71      +19     
- Misses        224      235      +11     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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