Skip to content

remove map_dfr - fixes posit-dev/connectapi#422#496

Open
AntLP wants to merge 2 commits intoposit-dev:mainfrom
AntLP:remove-map_dfr-usage
Open

remove map_dfr - fixes posit-dev/connectapi#422#496
AntLP wants to merge 2 commits intoposit-dev:mainfrom
AntLP:remove-map_dfr-usage

Conversation

@AntLP
Copy link

@AntLP AntLP commented Feb 6, 2026

Intent

As described in the issue, some functions had a dependency on dplyr through the use of purrr::map_df and purrr::map_dfr.

Fixes #422

Approach

Replaced the calls to purrr::map_df and purrr::map_dfr with a combination of purrr::map and purrr::list_rbind, which removes the dependency on dplyr.

I did not remove the purrr::map_dfr usages in the tests, dplyr is therefore still required to run the tests.

Checklist

  • Does this change update NEWS.md (referencing the connected issue if necessary)?
  • Does this change need documentation? Have you run devtools::document()?

@AntLP AntLP marked this pull request as ready for review February 6, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove usage of purrr::map_dfr to remove implicit dplyr dependency

1 participant