search

tidyverse / magrittr

Improve the readability of R code with the pipe
https://magrittr.tidyverse.org
Other
959 stars 157 forks source link

Fix visibility forwarding of `%$%` on old R versions #227

Closed lionel- closed 4 years ago

lionel- commented 4 years ago

By making it follow the lazy path. Closes #226. It failed because the eager code path cleans up .. On older R versions, this calls back into the R-level rm() function which returns invisibly. On 4.0 there was no issue because we delete the binding with the new C-level API. This is worked around by following the lazy path which is more appropriate anyway.

Also make lazy the other pipes %T> and %<>%.