insightsengineering / insightsengineering/teal.transform

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

Open
#191 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

`{rlang}` is used just a couple of times in 3 cases

- `rlang::is_empty` -> substitute with `length(x) == 0`
- `rlang::set_names` -> substitute with `stats::setNames`.
- Documentation of rlang::set_names` states: This is equivalent to `stats::setNames()`, with more features and stricter argument checking.
- `rlang::parse_expr` -> substitute with `base::parse`.
- Documentation of `rlang::parse_expr`states they are barely the same and the only difference is that: `parser_expr`, unlike `base::parse`, never retain source reference information, as doing so is slow and rarely necessary. We can substitute with `parser(keep.source = FALSE)`.

Consider if we can substitute those with equivalents from `{base}` package.

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.