Add support for multiple predicates to is.any
#104
Conversation
readme.md
Outdated
| @@ -432,6 +432,20 @@ is.any(is.boolean, 'unicorns', [], new Map()); | |||
| //=> false | |||
| ``` | |||
|
|
|||
| See below for use with multiple predicates. | |||
There was a problem hiding this comment.
If it's preferred, I can document both usages in the same block, perhaps like .any(predicate|predicates[], ...values)
There was a problem hiding this comment.
I see a thumbs-up, should I put them in the same block?
There was a problem hiding this comment.
Actually, I just saw that you separated single and multiple predicates to different blocks. I think it's better. Keep it as is
There was a problem hiding this comment.
Oh sorry, I updated it before seeing this latest comment. Please let me know what I should do.
sindresorhus
changed the title
is.any
scraggo
force-pushed
the
add-multiple-predicates-to-any
branch
from
1a61b0e
to
b662c7c
Compare
readme.md
Outdated
| @@ -420,9 +420,9 @@ Returns `true` if `value` is an even integer. | |||
|
|
|||
| Returns `true` if `value` is an odd integer. | |||
|
|
|||
| ##### .any(predicate, ...values) | |||
| ##### .any(predicate | predicates[], ...values) | |||
There was a problem hiding this comment.
predicates[] -> predicate[], they're the same type.
readme.md
Outdated
| @@ -432,6 +432,16 @@ is.any(is.boolean, 'unicorns', [], new Map()); | |||
| //=> false | |||
| ``` | |||
|
|
|||
| Using an array of `predicates`, returns `true` if **any** of the input `values` returns true for **any** of the `predicates` provided in an array: | |||
There was a problem hiding this comment.
"Using an array of predicate[]..."
|
Thanks @scraggo! 🎉 |
|
Thanks for letting me contribute! 😄 |
Fixes #98
Open to all commentary, especially on type declarations!