Merge branch 'main' into jomain

rformassspectrometry · Oct 16, 2024 · 6344dd5 · 6344dd5
2 parents 88524bd + 1400f6f
commit 6344dd5
Showing 1 changed file with 34 additions and 33 deletions.
diff --git a/.github/workflows/check-bioc.yml b/.github/workflows/check-bioc.yml
@@ -53,7 +53,7 @@ jobs:
       fail-fast: false
       matrix:
         config:
-          - { os: ubuntu-latest, r: '4.4', bioc: '3.20', cont: "bioconductor/bioconductor_docker:devel", rspm: "https://packagemanager.rstudio.com/cran/__linux__/focal/latest" }
+          - { os: ubuntu-latest, r: 'devel', bioc: 'devel', cont: "bioconductor/bioconductor_docker:devel", rspm: "https://packagemanager.rstudio.com/cran/__linux__/focal/latest" }
           - { os: macOS-latest, r: '4.4', bioc: '3.20'}
           - { os: windows-latest, r: '4.4', bioc: '3.20'}
     env:
@@ -124,31 +124,32 @@ jobs:
       - name: Install macOS system dependencies
         if: matrix.config.os == 'macOS-latest'
         run: |
-          ## Enable installing XML from source if needed
-          brew install libxml2
-          echo "XML_CONFIG=/usr/local/opt/libxml2/bin/xml2-config" >> $GITHUB_ENV
-
-          ## Required to install magick as noted at
-          ## https://github.com/r-lib/usethis/commit/f1f1e0d10c1ebc75fd4c18fa7e2de4551fd9978f#diff-9bfee71065492f63457918efcd912cf2
-          brew install imagemagick@6
-
-          ## For textshaping, required by ragg, and required by pkgdown
-          brew install harfbuzz fribidi
-
-          ## For installing usethis's dependency gert
-          brew install libgit2
-
-          ## required for ncdf4 - can not use the homebrew one because that uses GCC
-          ## Use pre-compiled libraries from https://mac.r-project.org/libs-4/
-          curl -O https://mac.r-project.org/libs-4/netcdf-4.7.4-darwin.17-x86_64.tar.gz
-          tar fvxzm netcdf-4.7.4-darwin.17-x86_64.tar.gz -C /
-          rm netcdf-4.7.4-darwin.17-x86_64.tar.gz
-          curl -O https://mac.r-project.org/libs-4/hdf5-1.12.0-darwin.17-x86_64.tar.gz
-          tar fvxzm hdf5-1.12.0-darwin.17-x86_64.tar.gz -C /
-          rm hdf5-1.12.0-darwin.17-x86_64.tar.gz
-          curl -O https://mac.r-project.org/libs-4/szip-2.1.1-darwin.17-x86_64.tar.gz
-          tar fvxzm szip-2.1.1-darwin.17-x86_64.tar.gz -C /
-          rm szip-2.1.1-darwin.17-x86_64.tar.gz
+          # ## Enable installing XML from source if needed
+          # brew install libxml2
+          # echo "XML_CONFIG=/usr/local/opt/libxml2/bin/xml2-config" >> $GITHUB_ENV
+
+          # ## Required to install magick as noted at
+          # ## https://github.com/r-lib/usethis/commit/f1f1e0d10c1ebc75fd4c18fa7e2de4551fd9978f#diff-9bfee71065492f63457918efcd912cf2
+          # brew install imagemagick@6
+
+          # ## For textshaping, required by ragg, and required by pkgdown
+          # brew install harfbuzz fribidi
+
+          # ## For installing usethis's dependency gert
+          # brew install libgit2
+
+          # ## required for ncdf4 - can not use the homebrew one because that uses GCC
+          # ## Use pre-compiled libraries from https://mac.r-project.org/libs-4/
+          # curl -O https://mac.r-project.org/libs-4/netcdf-4.7.4-darwin.17-x86_64.tar.gz
+          # tar fvxzm netcdf-4.7.4-darwin.17-x86_64.tar.gz -C /
+          # rm netcdf-4.7.4-darwin.17-x86_64.tar.gz
+          # curl -O https://mac.r-project.org/libs-4/hdf5-1.12.0-darwin.17-x86_64.tar.gz
+          # tar fvxzm hdf5-1.12.0-darwin.17-x86_64.tar.gz -C /
+          # rm hdf5-1.12.0-darwin.17-x86_64.tar.gz
+          # curl -O https://mac.r-project.org/libs-4/szip-2.1.1-darwin.17-x86_64.tar.gz
+          # tar fvxzm szip-2.1.1-darwin.17-x86_64.tar.gz -C /
+          # rm szip-2.1.1-darwin.17-x86_64.tar.gz
+        shell: Rscript {0}
 
       - name: Install Windows system dependencies
         if: runner.os == 'Windows'
@@ -196,12 +197,12 @@ jobs:
           remotes::install_local(dependencies = TRUE, repos = BiocManager::repositories(), build_vignettes = FALSE, upgrade = TRUE)
 
           ## Manually install required packages
-          BiocManager::install('rhdf5', dependencies = TRUE, ask = FALSE, update = FALSE, INSTALL_opts = '--force-biarch')
-          BiocManager::install("mzR", dependencies = TRUE, ask = FALSE, update = FALSE)
-          BiocManager::install("msdata")
-          BiocManager::install("HenrikBengtsson/matrixStats")
-          BiocManager::install("SummarizedExperiment", dependencies = TRUE, ask = FALSE, update = FALSE)
-          BiocManager::install("QFeatures", dependencies = TRUE, ask = FALSE, update = FALSE)
+          # BiocManager::install('rhdf5', dependencies = TRUE, ask = FALSE, update = FALSE, INSTALL_opts = '--force-biarch')
+          # BiocManager::install("mzR", dependencies = TRUE, ask = FALSE, update = FALSE)
+          # BiocManager::install("msdata")
+          # BiocManager::install("HenrikBengtsson/matrixStats")
+          # BiocManager::install("SummarizedExperiment", dependencies = TRUE, ask = FALSE, update = FALSE)
+          # BiocManager::install("QFeatures", dependencies = TRUE, ask = FALSE, update = FALSE)
           BiocManager::install("AnnotationHub", dependencies = TRUE, ask = FALSE, update = FALSE)
           BiocManager::install("RforMassSpectrometry/CompoundDb")
           library(AnnotationHub)
@@ -282,7 +283,7 @@ jobs:
       - name: Test coverage
         if: github.ref == 'refs/heads/main' && env.run_covr == 'true' && runner.os == 'Linux'
         run: |
-          covr::codecov()
+          covr::codecov(token = "{{ secrets.CODECOV_TOKEN }}")
         shell: Rscript {0}
 
       - name: Install package