search

epiverse-trace / cleanepi

R package to clean and standardize epidemiological data
https://epiverse-trace.github.io/cleanepi/
Other
8 stars 3 forks source link

Switch to base R pipe #147

Closed Bisaloo closed 5 months ago

Bisaloo commented 5 months ago

We're now allowed to depend on R (>=4.0.0) as per our (and the tidyverse's) policy.

It would be a good opportunity to convert the magrittr pipes to base R pipes and get rid of the downsides linked with magrittr pipes (more difficult to debug in particular).

sbfnk commented 5 months ago

For the base R pipe wouldn't we need to depend on >= 4.1.0?

Bisaloo commented 5 months ago

Oh yes, you're completely right. For some reason, I had in mind that the pipe was introduces in 4.0.0 and lambda functions in 4.1.0 but it looks that both were indeed introduced in 4.1.0 :exploding_head: