Skip to content

support --coverage-filter flag #60

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 4, 2024
Merged

support --coverage-filter flag #60

merged 1 commit into from
Mar 4, 2024

Conversation

shish
Copy link
Contributor

@shish shish commented Feb 26, 2024

Since phpunit 9, --whitelist got replaced with --coverage-filter

Also, since any type of coverage requires a filter, and also requires XDEBUG_MODE=coverage, I have made the flag automatically set the environment variable so that users don't need to figure that out for themselves

@shish shish marked this pull request as draft February 26, 2024 21:17
@shish shish marked this pull request as ready for review February 26, 2024 21:21
@g105b
Copy link
Member

g105b commented Feb 27, 2024

Thank you for this @shish , I'll give it a test and then we can merge it in.

@shish
Copy link
Contributor Author

shish commented Mar 2, 2024

@g105b
Copy link
Member

g105b commented Mar 4, 2024

Brilliant work, thank you!

@g105b g105b merged commit d1db433 into php-actions:master Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants