-
-
Notifications
You must be signed in to change notification settings - Fork 19
Added PHPUnit 8 support #31
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
Added PHPUnit 8 support #31
Conversation
I guess |
@OndraM Yes, but the changes we need to do in https://github.com/php-mock/php-mock, and there we still support PHP 5.6. I can prepare compatibility level to support both versions, but not sure if it is worth to do that. |
Well I don't think its worth the effort: you can just drop the support of old PHP - who uses it can still use the old version of this library, it won't stop working for him. But anyone with PHPUnit 6+ must already use PHP 7+ (see https://phpunit.de/supported-versions.html ). |
I agree with @OndraM . I think would be better release major version with dropped php5. In readme should point which version should use. PS |
I've decided to add PHPUnit 8 support without new major release. @KartaviK @OndraM Please have a look on PR: php-mock/php-mock#22 if you have a sec. Thanks! |
InvocationMocker is marked as final and cannot be mocked anymore.
No description provided.