Skip to content
This repository has been archived by the owner on Oct 17, 2021. It is now read-only.

Migrate from base to tidyverse #41

Open
Guillawme opened this issue Feb 12, 2018 · 2 comments
Open

Migrate from base to tidyverse #41

Guillawme opened this issue Feb 12, 2018 · 2 comments

Comments

@Guillawme
Copy link
Owner

Guillawme commented Feb 12, 2018

This is mostly polishing.

All occurrences of lapply and mapply could be rewritten using map functions from the purrr package.

@Guillawme
Copy link
Owner Author

Also, use stringr functions instead of base function at the few places that have string operations.

Guillawme added a commit that referenced this issue Mar 14, 2018
I still need to replace some occurrences of mapply, but I don't know how to deal with nested purrr::map (whereas it works with mapply).
@Guillawme
Copy link
Owner Author

There are still a couple occurrences of nested mapply calls, which I don't know yet how to properly translate to map.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant