Skip to content

Collect coverage with Xdebug when the locked php-code-coverage cannot use PCOV - #317

Merged
schlessera merged 1 commit into
mainfrom
ci/coverage-driver-for-locked-code-coverage
Nov 2, 2026
Merged

schlessera merged 1 commit into
mainfrom
ci/coverage-driver-for-locked-code-coverage

Conversation

@schlessera

Copy link
Copy Markdown
Member

Since d96f31a the Behat coverage jobs default to PCOV. PCOV needs phpunit/php-code-coverage 9.1 or later, which picks the loaded driver through its Selector. Older versions only use Xdebug. That's what wp-cli-tests falls back to, and it dies with This driver requires Xdebug in every scenario.

Packages without a composer.lock install a current php-code-coverage and aren't affected. wp-cli-bundle locks 7.0.17 because its platform is PHP 7.2, so every bundle PR has failed its coverage jobs since (e.g. wp-cli/wp-cli-bundle#1202).

The matrix step now reads the locked php-code-coverage version. If it's older than 9.1, coverage entries default to coverage_driver: xdebug. An entry that sets coverage_driver itself keeps it.

🤖 Generated with Claude Code

… use PCOV

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@schlessera
schlessera requested a review from a team as a code owner November 2, 2026 16:10
@coderabbitai

coderabbitai Bot commented Nov 2, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0956112c-eafe-489d-9d04-b34bf4c38095
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

schlessera added a commit to wp-cli/wp-cli-bundle that referenced this pull request Nov 2, 2026
@github-actions github-actions Bot added bug Something isn't working scope:testing labels Nov 2, 2026
@schlessera
schlessera merged commit dde70b8 into main Nov 2, 2026
19 checks passed
@schlessera
schlessera deleted the ci/coverage-driver-for-locked-code-coverage branch November 2, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working scope:testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant