BAAQMD / BAAQMD/geotools

Switch from lazyeval to rlang

Open
#3 0 comments 0 reactions 2 assignees Claimed by @dholstius 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._

`geotools` has one usage of `lazyeval`:
```
filter_.SpatialDataFrame <- function (.data, ..., .dots) {
dots <- lazyeval::all_dots(.dots, ..., all_named = TRUE)
masks <- lazyeval::lazy_eval(dots, data = as.data.frame(.data@data))
subset(.data, Reduce(`&&`, masks))
}
```
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

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.