BAAQMD / BAAQMD/funtools

Migrate from lazyeval to rlang

Open
#4 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
R
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

As part of [Resolve namespace conflicts among packages attached by library(inventory)](https://github.com/BAAQMD/inventory/issues/19) we would like to remove all occurences of `lazyeval` in the codebase and replace them with rlang. According to RStudio: _lazyeval has been superseded by rlang. We're working on better communicating these changes to the user with deprecation messages using lifecycle._

`funtools` has one usage of `lazyeval`:
```
pmean <- function (..., na.rm = TRUE) {
dots <- lazyeval::lazy_dots(...)
args <- lazyeval::lazy_eval(dots)
f <- function (...) base::mean(c(...), na.rm = na.rm)
purrr::pmap_dbl(args, f)
}
```
We should find a way to convert this code to use rlang, so that we can remove the package completely from our setup.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the codebase and package setup for `lazyeval`, then inspect the `pmean` entry point shown in the issue. Compare the existing lazy argument handling with the relevant rlang approach and check any package tests or checks. Done means the usage is migrated and no `lazyeval` dependency or occurrence remains.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
tooling
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.