-
Notifications
You must be signed in to change notification settings - Fork 29
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
Support PHPUnit 10 #64
Comments
Regarding the last point, to be clear: |
Well, the last point is not the hard block. Migrating to the new method will be easy. But it is not high priority as long as the other points are not resolved. |
I'll consider jumping into some of those tasks. On a totally unrelated sidenote, there are plenty of finished things ready to be merged (some of which are blocking the classic-webdriver) - what's the situation in general? Is it ok to merge my own PR given review approval? |
@stof , The issue was merged. Maybe we can proceed with PHPUnit 10 support now? |
this is a meta-issue tracking blockers for using PHPUnit 10:
yoast/phpunit-polyfills
(at least marking it as compatible in the composer constraint)TestCase::onNotSuccessfulTest
to mark tests as skipped as this is not supported anymoreTestCase::getName
was removed, replacing it with 2 methods instead of having a boolean flag)The text was updated successfully, but these errors were encountered: