Adds tidyr as a suggests, disables lintr

chendaniely · Dec 16, 2019 · d818814 · d818814
1 parent 6a6b21c
commit d818814
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/.travis.yml b/.travis.yml
@@ -56,4 +56,4 @@ r_github_packages:
 
 after_success:
   - Rscript -e 'covr::codecov()'
-  - Rscript -e 'lintr::lint_package()'
+#  - Rscript -e 'lintr::lint_package()'
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -35,6 +35,8 @@ Imports:
     glue,
     checkmate,
     testthat
+Suggests:
+    tidyr
 Remotes:
     rstudio/learnr
 URL: https://rstudio-education.github.io/gradethis/,

diff --git a/tests/testthat/test_check_code.R b/tests/testthat/test_check_code.R
@@ -209,6 +209,8 @@ test_that("Spot differences when pipes are involved", {
 test_that("Spots differences in long calls", {
   # original discussion here:
   # https://github.com/rstudio-education/grader/issues/28
+
+  testthat::skip_if_not_installed("tidyr")
 
   user <- quote(tidyr::gather(key = key, value = value, new_sp_m014:newrel_f65, na.rm = TRUE)) # nolint
   solution <- quote(tidyr::gather(key = key, value = value, new_sp_m014:newrel_f65, na.rm = FALSE)) # nolint