-
-
Notifications
You must be signed in to change notification settings - Fork 205
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
refactor: use %||% from rlang #1112
Conversation
Current Aviator status
This PR was merged using Aviator. See the real-time status of this PR on the Aviator webapp. Use the Aviator Chrome Extension to see the status of your PR within GitHub.
|
@krlmlr thanks for reporting r-lib/roxygen2#1570, seeing it made me fix the problem much quicker 😅 |
} | ||
} | ||
} | ||
|
||
`%&&%` <- function(lhs, rhs) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@krlmlr does this operator exist in other packages? even as utility, I had never seen it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
} | ||
} | ||
|
||
`%&&%` <- function(lhs, rhs) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
36171e0
to
4e04d26
Compare
later this will be from base R itself 😁
I noticed this function's name in the "sanitizer" workflow results and while looking for occurrences of it in the codebase, I saw it was home-made.