Merge pull request #236 from pharmaverse/221-swap-in-lintrlibrary_call_linter

Closes #221

Author: kodesiba

DESCRIPTION

@@ -65,7 +65,7 @@ Suggests:
     pkgdown,
     Tplyr,
     haven,
-    lintr,
+    lintr (>= 3.1.1),
     xml2,
     here,
     readr,

NEWS.md

@@ -1,3 +1,7 @@
+# logrx 0.3.3
+
+ - Swap in `lintr::library_call_linter()` for local version of `library_call_linter()` (#221)
+
 # logrx 0.3.2
 
  - Hotfix to update tests (#231)

R/library_call_linter.R

@@ -172,7 +172,7 @@ test_that("library lint returns expected result when multiple linters are set",
    skip_if_not_installed("xml2")
 
    options("log.rx" = NULL)
-   withr::local_options(log.rx.lint = c(library_call_linter(), lintr::undesirable_operator_linter()))
+   withr::local_options(log.rx.lint = c(lintr::library_call_linter(), lintr::undesirable_operator_linter()))
    filename <- test_path("ref", "ex7.R")
 
    # get is called within log_config

tests/testthat/Rplots.pdf

@@ -223,7 +223,7 @@ test_that("write_lint_results works when linter is used but no lints found", {
 
    options("log.rx" = NULL)
    log_config(filename)
-   lint_results <- lintr::lint(filename, c(library_call_linter()))
+   lint_results <- lintr::lint(filename, c(lintr::library_call_linter()))
    assign('lint_results', lint_results, envir = getOption('log.rx'))
 
    expect_identical(

tests/testthat/test-get.R

@@ -51,7 +51,7 @@ If you or your organization would like to implement any linters, you can set you
 
 <br>
 
-It is recommended to use the `library_call_linter()`.  This is to ensure `logrx` will find the correct package and functions used.
+It is recommended to use the `lintr::library_call_linter()`.  This is to ensure `logrx` will find the correct package and functions used.
 
 <br>