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
Add PHPUnit 9 support #22
Conversation
…d memory for this case.
@steve-todorov Thanks for your PR! I fixed the issue with CI not being hooked to PRs. There seems to be an issue with PHP 7.2 builds. Since apparently PHPUnit dropped support for PHP <= 7.2, I'll fix the expected versions in our TravisCI config file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Thank you!
I'm ready to merge this, unless you have any concerns @hrcc ? |
Thanks for fixing the build! I haven't updated the |
Looks good, thanks @steve-todorov! |
Awesome! Thanks for releasing! :) |
This PR fixes #21 and adds support for PHPUnit 9.
A few key notes:
ResultPrinter
is now an interface andDefaultResultPrinter
should be extended instead. (Make PHPUnit\TextUI\ResultPrinter an interface sebastianbergmann/phpunit#4024)phpunit
to 9 and fixed a few warnings.