-
Notifications
You must be signed in to change notification settings - Fork 38
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
Export where()
#201
Comments
This is on purpose, for now. |
Ah gotcha. Is there a reason for this? I currently have functions that have defaults like |
Just hesitance regarding using up a good function name. I think we'll end up exporting it anyway, since it'd probably be confusing to have different functionality for the same symbol. R CMD check note is a good point (altough easy to work around). |
E.g., assume it is used inside a
How should we use |
|
I just ran into the same issue with re-exporting |
I managed to make it work by doing:
|
It was missing add tidyselect.
@lhmet I would really recommend this instead: utils::globalVariables("where")
|
Any idea when you will export where? |
Probably in the next version, but that's not high priority because the workaround is easy and non-invasive. |
Writing my first package and still struggling with where() even after reading this thread. I added this line into the function to define where, and then later call where(!is.list) in the function and it passes the CMD check fine. But when I run the package, I get an error.
Any ideas or updates on when we can just put tidyselect::where() into packages? |
Hello. You shouldn't assign where <- utils::globalVariables("where") Into: utils::globalVariables("where") I don't know why you're getting this error message though. Where did you put the |
@NateNohling @lionel- I ran into the same error because I put |
@petrbouchal Ah...got it! That fixed it! Thanks |
…ion that seems to be an ongoing github issue: r-lib/tidyselect#201
How does one actually call this function inside a package once it is OK'd with |
|
Just coming here to add to the call for exporting |
Idem. |
Idem :) |
So per @petrbouchal 's link: add Edit: this didn't actually fix it. So if anyone knows how to do so, please say. I'm still getting:
(gbm.auto is my package name) idem / +1 / samesies for exporting please. Cheers! Edit2: Spacedman & DHW note that the
but that doesn't work for me either. |
It is likely that |
I haven't been able to replace
|
@jjesusfilho would discard() work? |
documented issue, r-lib/tidyselect#201
documented issue, r-lib/tidyselect#201
I was trying to reexport
where()
for my package that uses tidyselect, but the function isn't currently exported in v1.1.0.The text was updated successfully, but these errors were encountered: