Recommend using all_of()
for looking up variables
#127
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
First step towards #76. For now this is a simple message:
We can move on to formally deprecating this kind of lookup in the next version.
I would like to refer to
all_of()
in the message, but there will be a transition period where dplyr hasn't exported the new selectors. To work around this, the second commit binds the selection helpers inside the context mask. This is what several client packages are already doing. This makes it easier to use dplyr functions without the package attached:It is still recommended to export the helpers so their help topics are easily accessible.