try pre-installing to speed this up

microsoft · Dec 2, 2024 · a43cb3a · a43cb3a
1 parent 321ce3c
commit a43cb3a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.ci/run-revdep-checks.sh b/.ci/run-revdep-checks.sh
@@ -8,9 +8,4 @@ mkdir -p "${CHECKS_OUTPUT_DIR}"
 sh build-cran-package.sh --no-build-vignettes
 mv ./lightgbm_*.tar.gz "${CHECKS_OUTPUT_DIR}/"
 
-# pre-install all of the dependencies... tools::check_packages_in_dir()
-# is hard-coded to compile them all from source
-# (https://github.com/wch/r-source/blob/594b842678e932088b16ec0cd3c39714a141eed9/src/library/tools/R/checktools.R#L295)
-Rscript ./.ci/download-r-revdeps.R
-
 Rscript ./.ci/run-r-revdepchecks.R "${CHECKS_OUTPUT_DIR}"
diff --git a/.github/workflows/r_revdepchecks.yml b/.github/workflows/r_revdepchecks.yml
@@ -57,8 +57,11 @@ jobs:
           Rscript -e "install.packages(c('R6', 'data.table', 'jsonlite', 'knitr', 'markdown', 'Matrix', 'RhpcBLASctl'), repos = 'https://cran.r-project.org', Ncpus = parallel::detectCores(), type = 'binary')"
           sh ./build-cran-package.sh
       - name: pre-install binary packages
+        # pre-install all of the dependencies... tools::check_packages_in_dir()
+        # is hard-coded to compile them all from source
+        # (https://github.com/wch/r-source/blob/594b842678e932088b16ec0cd3c39714a141eed9/src/library/tools/R/checktools.R#L295)
         run: |
-          Rscript ./.ci/download-r-revdeps.R
+          Rscript --vanilla ./.ci/download-r-revdeps.R
       - name: run revdepchecks
         run: |
           bash ./.ci/run-revdep-checks.sh