databricks-sdk-r icon indicating copy to clipboard operation
databricks-sdk-r copied to clipboard

Switch out dplyr dependency for vctrs

Open hadley opened this issue 2 years ago • 3 comments

dplyr is a user facing package so it's rather dependency heavy in favour of providing the whole package to users. This means it doesn't feel like a great dependency for an SDK, which wants to be low-level. Fortunately, it should be pretty easy to switch out dplyr for vctrs, which is the low-level equivalent, replacing dplyr::bind_rows() with vctrs::vec_rbind().

hadley avatar Jun 23 '23 19:06 hadley

@hadley what license does vctrs package have? I support the idea of using as little transitive dependencies as possible, though we have to go through approvals for any new packages. e.g. I had to remove configr dep for manual parting of INI file for this same purpose.

nfx avatar Jun 24 '23 15:06 nfx

It’s MIT.

hadley avatar Jun 24 '23 17:06 hadley

should be easy to get through, then

nfx avatar Jun 24 '23 19:06 nfx