jonkeane commented on a change in pull request #11306:
URL: https://github.com/apache/arrow/pull/11306#discussion_r724237701
##########
File path: r/R/util.R
##########
@@ -66,6 +66,11 @@ r_symbolic_constants <- c(
)
is_function <- function(expr, name) {
+ # We could have a quosure here if we have an expression like
+ # `sum({{ var }})`
Review comment:
There are a few options: having more words in front of the code or `#
nolint` at the end. We can also turn this one particular lintr off (or it's
possible there are settings for it that makes it less sensitive) by adding
`commented_code_linter = NULL` to
https://github.com/apache/arrow/blob/master/r/.lintr#L25
I've been meaning to send a PR to lintr to make backticks like this defuse
commented code — though their track record of merging these has been spotty so
I haven't made it a priority.
--
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]
