search

rstudio / pins-r

Pin, discover, and share resources
https://pins.rstudio.com
Other
312 stars 63 forks source link

Actually import purrr #722

Closed juliasilge closed 1 year ago

juliasilge commented 1 year ago

This PR sets up pins to use purrr directly, instead of using the older compat-purrr.R file (which is very out of date, doesn't have adverbs like possibly(), etc).

juliasilge commented 1 year ago

We were at 20 imported packages, so the first step I took here was to move several packages to Suggests that are only used in one function or only in legacy functions.

juliasilge commented 1 year ago

Not sure if you want to take a look at this or not @hadley -- let me know if not!

juliasilge commented 1 year ago

I don't think this specific PR uses any purrr features, but I presume you have some plans further down the line.

Yep, that's right. I've been running into some annoyances because of not having real purrr and this particular PR was finally motivated by me trying to fix #606 using possibly().

github-actions[bot] commented 1 year ago

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.