From 7a0ecd6c3b1aabd389ab27fd7c721b76e95e5ca6 Mon Sep 17 00:00:00 2001
From: "dr. M.S. (Matthijs) Berends"
 <31037261+msberends@users.noreply.github.com>
Date: Sun, 3 Oct 2021 23:49:46 +0200
Subject: [PATCH] No vignettes on old R unit tests

---
 .github/workflows/check.yaml | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/check.yaml b/.github/workflows/check.yaml
index 64b8f0d1..b2842431 100644
--- a/.github/workflows/check.yaml
+++ b/.github/workflows/check.yaml
@@ -119,17 +119,15 @@ jobs:
           as.data.frame(utils::installed.packages())[, "Version", drop = FALSE]
         shell: Rscript {0}
         
-      # # - name: Only keep vignettes on release version
-      # - name: Remove vignettes
-      #   # if: matrix.config.r != 'release'
-      #   if: always()
-      #   # writing to DESCRIPTION2 and then moving to DESCRIPTION is required for R < 3.3 as writeLines() cannot overwrite
-      #   run: |
-      #     rm -rf AMR/vignettes
-      #     Rscript -e "writeLines(readLines('AMR/DESCRIPTION')[!grepl('VignetteBuilder', readLines('AMR/DESCRIPTION'))], 'AMR/DESCRIPTION2')"
-      #     rm AMR/DESCRIPTION
-      #     mv AMR/DESCRIPTION2 AMR/DESCRIPTION
-      #   shell: bash
+      - name: Remove vignettes on R without knitr support
+        if: matrix.config.r == '3.0' || matrix.config.r == '3.1' || matrix.config.r == '3.2'
+        # writing to DESCRIPTION2 and then moving to DESCRIPTION is required for R < 3.3 as writeLines() cannot overwrite
+        run: |
+          rm -rf AMR/vignettes
+          Rscript -e "writeLines(readLines('AMR/DESCRIPTION')[!grepl('VignetteBuilder', readLines('AMR/DESCRIPTION'))], 'AMR/DESCRIPTION2')"
+          rm AMR/DESCRIPTION
+          mv AMR/DESCRIPTION2 AMR/DESCRIPTION
+        shell: bash
         
       - name: Run R CMD check
         if: always()