Exclude packrat and renv from linting as well (#42)

jumpingrivers · Apr 10, 2021 · 2ecaac0 · 2ecaac0
1 parent 6c7d0c4
commit 2ecaac0
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,7 +1,7 @@
 Type: Package
 Package: inteRgrate
 Title: Opinionated Package Coding Styles
-Version: 1.0.16
+Version: 1.0.17
 Authors@R:
     person(given = "Jumping",
            family = "Rivers",

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,6 @@
+# inteRgrate 1.0.17 _2021-04-10_
+  * Add more exclusions (again)
+
 # inteRgrate 1.0.16 _2021-04-07_
   * Fix exclusions parsing (again!)
 

diff --git a/R/check_lintr.R b/R/check_lintr.R
@@ -1,17 +1,17 @@
 get_exclusions = function() {
-  if (!file.exists(".lintr")) return(c("^R/|^cache/"))
+  if (!file.exists(".lintr")) return(c("^R/|^cache/|^packrat/|^renv/"))
 
   exclusions = read.dcf(".lintr", all = TRUE)$exclusions
 
-  if (is.null(exclusions)) return(c("^R/|^cache/"))
+  if (is.null(exclusions)) return(c("^R/|^cache/|^packrat/|^renv/"))
   exclusions = stringr::str_split(exclusions, ",")[[1]]
   exclusions = stringr::str_remove(exclusions, "list\\W?\\(")
   exclusions = stringr::str_remove(exclusions, "\\)$")
   exclusions = stringr::str_squish(exclusions)
   exclusions = stringr::str_remove_all(exclusions, ("^\"|\"$"))
 
   ## Convert to regular expression
-  pattern = paste0(c(exclusions, "^R/", "^cache/"), collapse = "|")
+  pattern = paste0(c(exclusions, "^R/", "^cache/", "^packrat/", "^renv/"), collapse = "|")
   return(pattern)
 }