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

Add new “any” test #4422

Closed
wants to merge 2 commits into from
Closed

Conversation

jdreesen
Copy link
Contributor

@jdreesen jdreesen commented Oct 28, 2024

PHP 8.4 will add the new array_find, array_find_key, array_any, and array_all functions.

A find filter was already added in #3963.

This PR adds the any test, which returns true if any of the elements in the array return true when passed to the arrow function.

@jdreesen
Copy link
Contributor Author

The fabbot compliant seems to be unrelated.

@fabpot
Copy link
Contributor

fabpot commented Oct 30, 2024

Isn't it the same as the current has some test?

@jdreesen
Copy link
Contributor Author

Oh wow, I completely missed that because I didn't look at the “Operators” section in the docs 🤦

Awesome! I'll close here.

@jdreesen jdreesen closed this Oct 30, 2024
@jdreesen jdreesen deleted the feat/any-filter branch October 30, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants