Copilot commented on code in PR #49708:
URL: https://github.com/apache/arrow/pull/49708#discussion_r3316665212
##########
r/R/dplyr-funcs-datetime.R:
##########
@@ -671,7 +671,7 @@ register_bindings_datetime_parsers <- function() {
notes = c(
"`quiet = FALSE` is not supported",
"Available formats are H, I, j, M, S, U, w, W, y, Y, R, T.",
- "On Linux and OS X additionally a, A, b, B, Om, p, r are available."
+ "On Linux and macOS additionally a, A, b, B, Om, p, r are available."
Review Comment:
This source note was updated to say macOS, but the generated documentation
included in the PR still says "OS X" in both r/R/dplyr-funcs-doc.R and
r/man/acero.Rd. Please regenerate/update the generated docs so the user-facing
documentation reflects this source change consistently.
##########
r/R/dplyr-funcs-datetime.R:
##########
@@ -724,7 +724,7 @@ register_bindings_datetime_parsers <- function() {
register_binding(
paste0("lubridate::", tolower(order)),
parser_map_factory(order),
- notes = "`locale` argument not supported"
+ notes = "`locale` argument not supported; see docs for
`parse_date_time()` mapping for supported formats"
Review Comment:
The source note here does not match the generated documentation added for
these parser bindings, which says "see `parse_date_time()` for supported
formats". Since r/R/dplyr-funcs-doc.R and r/man/acero.Rd are generated from
these notes, please regenerate the docs from the final source wording or update
the source to match the generated output.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
