-
Notifications
You must be signed in to change notification settings - Fork 216
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 8.1 compatibility #209
Comments
https://github.com/danielstjules/Stringy/network/dependents
Last commit in 3.x (master) branch happened more than 4 years ago. It is interesting that the PHP 8.1 compatibility issue just come up now. |
PR will be coming soon. 🧑🏭 |
Sorry. I just noticed that there is an open PR which is related to PHP 8.1 compatibility. #208 Before I opened this issue, I had searched for an existing issue, but I forgot to look through the pull requests. |
….4, 8.0, 8.1 to Travis matrix
….4, 8.0, 8.1 to Travis matrix
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
…workflows/qa.xml
Hello @danielstjules A little attention here, please. 🙏 |
It's funny to think such package still exist on packagist. Creating an open source library then suddenly ignoring it leaving thousands of projects broken and outdated lol. |
+1 on this ! |
Deprecated: Return type of
Stringy\Stringy::offsetSet($offset, $value)
should either be compatible withArrayAccess::offsetSet(mixed $offset, mixed $value): void
, or the#[\ReturnTypeWillChange]
attribute should be used to temporarily suppress the notice in /.../vendor/danielstjules/stringy/src/Stringy.php on line 893The solution could be very similar to this: https://github.com/cebe/php-openapi/pull/143/files
Return type of
Stringy\Stringy::count()
should either be compatible withCountable::count(): int
, or the#[\ReturnTypeWillChange]
attribute should be used to temporarily suppress the notice.The text was updated successfully, but these errors were encountered: