[r/ci] Continue #2275 (#2395)

single-cell-data · Apr 5, 2024 · f1fb30b · f1fb30b
1 parent d2b15c2
commit f1fb30b
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 7 deletions.
diff --git a/.github/workflows/r-ci.yml b/.github/workflows/r-ci.yml
@@ -70,6 +70,13 @@ jobs:
       #
       # Please see https://github.com/single-cell-data/TileDB-SOMA/wiki/Branches-and-releases which
       # is crucial for anyone doing releases of TileDB-SOMA.
+      #
+      # Please edit both files in the same way:
+      # * r-ci.yml
+      # * r-python-interop-testing.yml
+      #
+      # Do not remove these comments until such time as we have eliminated our dependency on
+      # the TileDB-R package.
 
       #- name: Install r-universe build of tiledb-r (macOS)
       #  if: ${{ matrix.os == 'macOS-latest' }}

diff --git a/.github/workflows/r-python-interop-testing.yml b/.github/workflows/r-python-interop-testing.yml
@@ -41,13 +41,30 @@ jobs:
       - name: MkVars
         run: mkdir ~/.R && echo "CXX17FLAGS=-Wno-deprecated-declarations -Wno-deprecated" > ~/.R/Makevars
 
-      - name: Install r-universe build of tiledb-r (macOS)
-        if: ${{ matrix.os == 'macOS-latest' }}
-        run: cd apis/r && Rscript -e "install.packages('tiledb', repos = c('https://eddelbuettel.r-universe.dev', 'https://cloud.r-project.org'))"
-
-      - name: Install r-universe build of tiledb-r (linux)
-        if: ${{ matrix.os != 'macOS-latest' }}
-        run: cd apis/r && Rscript -e "options(bspm.version.check=TRUE); install.packages('tiledb', repos = c('https://eddelbuettel.r-universe.dev/bin/linux/jammy/4.3/', 'https://cloud.r-project.org'))"
+      # Uncomment these next two stanzas as needed whenever we've just released a new tiledb-r for
+      # which source is available but CRAN releases (and hence update r2u binaries) are not yet:
+      #
+      # IMPORTANT: these two stanzas should remain uncommented _only_ during the propagation time
+      # between (a) publication of source for a new TileDB-R _upon which_ TileDB-SOMA depends in its
+      # apis/R/DESCRIPTION file and (b) appearance of binaries.
+      #
+      # Please see https://github.com/single-cell-data/TileDB-SOMA/wiki/Branches-and-releases which
+      # is crucial for anyone doing releases of TileDB-SOMA.
+      #
+      # Please edit both files in the same way:
+      # * r-ci.yml
+      # * r-python-interop-testing.yml
+      #
+      # Do not remove these comments until such time as we have eliminated our dependency on
+      # the TileDB-R package.
+
+      #- name: Install r-universe build of tiledb-r (macOS)
+      #  if: ${{ matrix.os == 'macOS-latest' }}
+      #  run: cd apis/r && Rscript -e "install.packages('tiledb', repos = c('https://eddelbuettel.r-universe.dev', 'https://cloud.r-project.org'))"
+
+      #- name: Install r-universe build of tiledb-r (linux)
+      #  if: ${{ matrix.os != 'macOS-latest' }}
+      #  run: cd apis/r && Rscript -e "options(bspm.version.check=TRUE); install.packages('tiledb', repos = c('https://eddelbuettel.r-universe.dev/bin/linux/jammy/4.3/', 'https://cloud.r-project.org'))"
 
       - name: Build and install libtiledbsoma
         run: sudo scripts/bld --prefix=/usr/local && sudo ldconfig