search

tidyverse / magrittr

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

Add test for pipe with lazy evaluation #210

Closed zeehio closed 4 years ago

zeehio commented 4 years ago

Closes #195. The test, inspired on @lionel-'s issue already passed on master. I bisected the issue and it was fixed on 565eb9a6111fa432ab33284d13d59405621926a3.

The test fails on the magrittr 1.5 source code as it is now available on CRAN, so maybe it's time for a new magrittr release that includes this bugfix?

I came across this issue through the stackoverflow question: https://stackoverflow.com/questions/61834248/r-piping-input-to-a-function-factory/61836228#61836228

lionel- commented 4 years ago

Thank you, this was fixed by #218