-
Notifications
You must be signed in to change notification settings - Fork 158
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.
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
R CMD check and no visible binding for global variable '.' #29
Comments
This is fine; but still I get annoyed somtimes.. I just toyed around with something; but it may be too weird. Essentially, I tried bundling the code in this gist (https://gist.github.com/smbache/a528d77f136a4bbf0c19) in a package to see if the compiler would complain. It didn't; and one of the functions will work at runtime ( |
Cool. Btw, did you try this with |
R CMD is fine with it; I'm not convinced yet that I am ;-) The <- tricks the check, but is never carried out. Rather delayedAssign does the assignment... |
Is this entirely a joke? . <- "Shut up" I'd really like to make the NOTEs go away…. |
Then just put this in the package and the NOTE goes away. You can change the exact text according to your mood. :) |
Or do the official |
That just seems more cumbersome: :) if(getRversion() >= "2.15.1") utils::globalVariables(c(".")) More seriously, it is indeed better, it just instructs the check tool to ignore the dot, instead of actually creating a global variable. |
The more you know! Thanks. |
this solution suggested here tidyverse/magrittr#29
Wondering if the
|
|
… of the code that use non-standard evaluation. See: - http://stackoverflow.com/questions/9439256/how-can-i-handle-r-cmd-check-no-visible-binding-for-global-variable-notes-when - tidyverse/magrittr#29 Also added line for collate: field
@gaborcsardi @jennybc I added a formula feature in this branch: https://github.com/smbache/magrittr/tree/simplified You could then do letters %>%
~gsub("[a-k]", "z", .) Small thing, but would make the checker happier.. |
…mming with the dot!!! -tidyverse/magrittr#29
This is more of a question than an issue. You are probably aware of
R CMD check
complaining about the dot. I could just ignore the complaints of course, but they sometimes signal real errors, so every time I see the NOTE in the output, I need to check.I just put this in the package:
Do you see any potential problems with this? It seems to me that it will not interfere with the pipes, but I don't know much about magrittr internals.
The text was updated successfully, but these errors were encountered: