insightsengineering / insightsengineering/teal.transform

[Feature Request]: remove `{tidyselect}` from `{teal.transform}`

Open
#190 0 comments 0 reactions 0 assignees View on GitHub
core enhancement
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`

image

Part of https://github.com/insightsengineering/teal.transform/issues/194

Contributor guide

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.