`summarize_range()` now also works with list of data frames - Githubissues

2DegreesInvesting / tiltIndicator

Indicators for the TILT project

https://2degreesinvesting.github.io/tiltIndicator/

GNU General Public License v3.0

1 stars 1 forks source link

`summarize_range()` now also works with list of data frames #764

Closed maurolepore closed 2 months ago

maurolepore commented 2 months ago

Relates to 2DegreesInvesting/tiltIndicatorAfter#214

This PR:

Makes summarize_range() polymorphic. It now accepts not only data frames but also lists.
Allows passing the col argument as a string rather than a symbol. This makes it easier to program with summarize_range().

TODO

[x] Link related issue/PR.
[x] Describe the goal of the PR. Avoid details that are clear in the diff.
[x] Mark the PR as draft.
[x] Include a unit test.
[x] Review your own PR in "Files changed".
[x] Ensure the PR branch is updated.
[x] Ensure the checks pass.
[x] Change the status from draft to ready.
[x] Polish the PR title and description.

EXCEPTIONS

[ ] Slide here any item that you intentionally choose to not do.
[ ] Assign a reviewer. I'll request a review where this function will be used rather than here where it's implemented.