-
Notifications
You must be signed in to change notification settings - Fork 38
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
PHP 7.x compatibility? #38
Comments
I am interested in this update. I want to use it at Yii 1.1 project. |
Hi. I'm not actively developing this anymore, but there is no reason not to at least make it compatible with 7.2. So, I'll see what I can do |
Thanks a lot @vernes you ve already done a great job, and since Yii, has been updated to support PHP 7.2 it would be great to see extensions that are widely used to be updated as well. Looking forward to hearing from you. |
Do you have any specific PHP7 compatibility errors? I checked it and it works fine on php 7.2. I don't want to introduce any breaking changes, this library is in maintenance mode and as for yii1.1, should not be used for new projects anyway. |
There aren’t indeed any specific 7.2 errors as I noticed as well, just a little curious if it would worth a) updating the exception declarence over the dummy method where email is stored as eml, |
Hi,
are there any upcoming changes regrding PHP 7.x compatibility? perhaps remove Exceptions and replace them with Throwable etc?
The text was updated successfully, but these errors were encountered: