Handle objects in {glue} string templates

Appsilon / box.linters

lintr-compatible linters for box modules in R

4 stars 0 forks source link

Closes #96

Description

Added function to get objects and functions inside glue { } strings.
Updated the unused family of functions to become glue-compatible

Does not handle an edge case described in glue unit tests:

test_that("glue works with complex expressions", {
  `foo}\`` <- "foo"

  expect_equal(glue("{
      {
        '}\\'' # { and } in comments, single quotes
        \"}\\\"\" # or double quotes are ignored
        `foo}\\`` # as are { in backticks
      }
  }"),
  `foo}\``)
})

https://github.com/tidyverse/glue/blob/9f88e2179fc7aa6ff11214271fd61bba169f925c/tests/testthat/test-glue.R#L76-L87

Definition of Done

[ ] The change is thoroughly documented.
[ ] The CI passes (R CMD check, linter, unit tests, spelling).
[ ] Any generated files have been updated (e.g. .Rd files with roxygen2::roxygenise())

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.36%. Comparing base (fb22413) to head (db2730f). Report is 2 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #97 +/- ## ========================================== + Coverage 98.31% 98.36% +0.05% ========================================== Files 20 21 +1 Lines 770 797 +27 ========================================== + Hits 757 784 +27 Misses 13 13 ```

Appsilon / box.linters