Deprecate callable Regex objects

Author: ararslan

NEWS.md

@@ -1069,6 +1069,8 @@ Deprecated or removed
 
   * `contains` has been deprecated in favor of a more general `isfound` function ([#26283]).
 
+  * `Regex` objects are no longer callable. Use `isfound` instead ([#26283]).
+
   * The methods of `range` based on positional arguments have been deprecated in favor of
     keyword arguments ([#25896]).
 

base/deprecated.jl

@@ -1457,6 +1457,10 @@ end
 # PR #26283
 @deprecate contains(haystack, needle) isfound(needle, haystack)
 @deprecate contains(s::AbstractString, r::Regex, offset::Integer) isfound(r, s, offset=offset)
+function (r::Regex)(s)
+    depwarn("`(r::Regex)(s)` is deprecated, use `isfound(r, s)` instead.", :Regex)
+    isfound(r, s)
+end
 
 # Issue #25786
 @deprecate_binding DevNull devnull

base/regex.jl

@@ -153,8 +153,6 @@ function isfound(r::Regex, s::SubString; offset::Integer=0)
                      r.match_data)
 end
 
-(r::Regex)(s) = isfound(r, s)
-
 """
     match(r::Regex, s::AbstractString[, idx::Integer[, addopts]])