Skip to content

Commit 03e12bc

Browse files
committed
Explicitly uninstall pandoc for pandocless tests
r-cran-rmarkdown now depends on pandoc. In general I think this is a good practice for a package manager (we also add pandoc to the run requirements of the r-rmarkdown conda-forge package for the same reason). But in order to confirm `R CMD check` can run without pandoc, I need to remove it. Can't use install.packages() because that is configured to use APT via {bpsm}
1 parent 9d05ab5 commit 03e12bc

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

.github/workflows/pandocless.yaml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,18 @@ jobs:
4545
r-cran-shiny \
4646
r-cran-testthat \
4747
r-cran-withr
48-
dpkg-query --list r-*
48+
dpkg-query --list r-*
49+
shell: sudo bash {0}
50+
# r-cran-rmarkdown now depends on pandoc. In general I think this is a
51+
# good practice for a package manager (we also add pandoc to the run
52+
# requirements of the r-rmarkdown conda-forge package for the same
53+
# reason). But in order to confirm `R CMD check` can run without pandoc, I
54+
# need to remove it.
55+
- name: Install {rmarkdown} without pandoc
56+
run: |
57+
apt-get remove --yes pandoc || true
58+
apt-get install --yes r-cran-remotes
59+
Rscript -e 'remotes::install_github("rstudio/rmarkdown@*release", dependencies = FALSE)'
4960
shell: sudo bash {0}
5061
- name: Session information
5162
run: sessionInfo()

0 commit comments

Comments
 (0)