switch to using .by for user argument for group variable

tidymodels / probably

Tools for post-processing class probability estimates

https://probably.tidymodels.org/

Other

111 stars 12 forks source link

switch to using .by for user argument for group variable #109

Closed EmilHvitfeldt closed 1 year ago

EmilHvitfeldt commented 1 year ago

To do change 2 and finally close https://github.com/tidymodels/probably/issues/104.

This PR changes the userfacing argument group to .by to align with {dplyr} syntax.

I'm keeping the internal arguments as group as I find it easier to read.

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.