Skip to content
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

Update Enumerable.php #63

Open
wants to merge 1 commit into
base: v2.0
Choose a base branch
from
Open

Update Enumerable.php #63

wants to merge 1 commit into from

Conversation

GarciaSiego
Copy link

Fix Deprecated: Return type of YaLinqo\Enumerable::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /vendor/athari/yalinqo/YaLinqo/Enumerable.php on line 50 1

Fix Deprecated: Return type of YaLinqo\Enumerable::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /vendor/athari/yalinqo/YaLinqo/Enumerable.php on line 50
1
@miszczu
Copy link

miszczu commented May 14, 2024

This project seems to be dead to be honest

@ekazda
Copy link

ekazda commented Sep 19, 2024

This project seems to be dead to be honest

@miszczu It would seem that you are right. Have you found a viable alternative? I have a project that still uses this and I'm likely going to have to upgrade before this warning becomes a real issue.

@miszczu
Copy link

miszczu commented Oct 3, 2024

@ekazda In my case it is used in a Laravel project, so we plan to switch to Illuminate\Support\Collection. Haven't look for other alternatives.

@ekazda
Copy link

ekazda commented Oct 3, 2024

@ekazda In my case it is used in a Laravel project, so we plan to switch to Illuminate\Support\Collection. Haven't look for other alternatives.

This is what I ended up doing also. Thanks for the confirmation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants