ianmcook commented on a change in pull request #10190:
URL: https://github.com/apache/arrow/pull/10190#discussion_r627605503
##########
File path: r/R/dplyr.R
##########
@@ -603,9 +603,16 @@ format_string_replacement <- function(replacement,
ignore.case, fixed) {
replacement
}
-# this function assigns definitions for the stringr pattern modifier functions
-# (fixed, regex, etc.) in itself, and uses them to evaluate the quoted
-# expression `pattern`
+#' Get `stringr` pattern options
+#'
+#' This function assigns definitions for the stringr pattern modifier functions
+#' (fixed, regex, etc.) in itself, and uses them to evaluate the quoted
+#' expression `pattern`
+#'
+#' @param pattern Unevaluated pattern to be evaluated
+#'
+#' @return List containing elements `pattern`, `fixed`, and `ignore_case`
+#' @keywords internal
Review comment:
Nice! Thanks for doing this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
