object_usage_linter() misses functions defined with \(x) lambda #1933

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.

Sign up for GitHub

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

Jump to bottom

Closed

MichaelChirico opened this issue Mar 25, 2023 · 0 comments · Fixed by #1934

Labels

false-negative object linters 🏀

Collaborator

MichaelChirico commented Mar 25, 2023

lintr::lint("f <- \\() x\n", lintr::object_usage_linter())
lintr::lint("f <- function() x\n", lintr::object_usage_linter())
# Warning message:
# In parse_check_usage(fun, known_used_symbols = known_used_symbols,  :
#   Possible bug in lintr: Couldn't parse usage message ‘<anonymous>: no visible binding for global variable 'x'
# ’. Ignoring 1 usage warnings.

(the warning in the second case is #1917)

MichaelChirico added object linters 🏀 false-negative labels

MichaelChirico mentioned this issue

detect functional lambdas in object_usage_linter #1934

Merged

IndrajeetPatil closed this as completed in #1934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment