insightsengineering / insightsengineering/teal.transform
[Feature Request]: remove `{tidyselect}` from `{teal.transform}`
- Dominant language
- R
- Stars
- 2
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
Consider if we can substitute `{tidyselect}` package usage with a function from `base`.
`{tidyselect}` is only used once `get_reshape_call()` in `R/get_dplyr_call.R`.
I don't think this function is even used in `teal.transform` or outside of `teal.transform`
Part of https://github.com/insightsengineering/teal.transform/issues/194
Contributor guide
Research direction
Start in R/get_dplyr_call.R, inspect get_reshape_call(), and search the package for references to that function and {tidyselect}. Check the package dependency declarations and existing checks. Done means {tidyselect} is no longer needed without breaking the package or its transformation behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100