-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 contains (argument flipped occursin) #35132
Conversation
Triage is ok with this. |
Is this ready for merge? |
bump |
1 similar comment
bump |
Thanks for this! |
Does this really a different function than in ? using Test
@test 6 |> in(1:10)
@test "JuliaLang" |> startswith(r"Julia|Romeo")
@test_broken "aba" |> in(r"a.a") # will throw
@test "aba" |> contains(r"a.a") Why do we not allow ? using Test
using Test
@test 6 |> in(1:10)
@test "JuliaLang" |> startswith(r"Julia|Romeo")
@test"aba" |> in(r"a.a") |
In is for elements, |
It is not in the essence of multidispatch to specialize function name according to type. Then we can talk for hours of what is a type. |
closes #35031
See issue for discussion.
Includes curried form per convention in #35052
in 2.0 we may want to remove
occursin