From 9fb2061220452200eaeeb5557178df5b80cda01d Mon Sep 17 00:00:00 2001 From: Hugo Gruson Date: Thu, 7 May 2026 15:11:11 +0200 Subject: [PATCH 1/4] Add action to check SpatialData.plot on changes to SpatialData --- .../R-CMD-check-SpatialDataPlot.yaml | 52 +++++++++++++++++++ 1 file changed, 52 insertions(+) create mode 100644 .github/workflows/R-CMD-check-SpatialDataPlot.yaml diff --git a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml new file mode 100644 index 00000000..bca3a906 --- /dev/null +++ b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml @@ -0,0 +1,52 @@ +# Workflow derived from https://github.com/r-lib/actions/tree/v2/examples +# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help +on: + push: + branches: [main, devel] + pull_request: + +name: R-CMD-check.yaml + +permissions: read-all + +jobs: + R-CMD-check: + runs-on: ${{ matrix.config.os }} + + name: ${{ matrix.config.os }} (${{ matrix.config.r }}) + + strategy: + fail-fast: false + matrix: + config: + - {os: macos-latest, r: 'release'} + - {os: windows-latest, r: 'release'} + - {os: ubuntu-latest, r: 'devel', http-user-agent: 'release'} + - {os: ubuntu-latest, r: 'release'} + + env: + GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }} + R_KEEP_PKG_SOURCE: yes + + steps: + - uses: actions/checkout@v6 + with: + repository: HelenaLC/SpatialData.Plot + + - uses: r-lib/actions/setup-pandoc@v2 + + - uses: r-lib/actions/setup-r@v2 + with: + r-version: ${{ matrix.config.r }} + http-user-agent: ${{ matrix.config.http-user-agent }} + + - uses: r-lib/actions/setup-r-dependencies@v2 + with: + extra-packages: any::rcmdcheck + needs: check + + - uses: r-lib/actions/check-r-package@v2 + with: + upload-snapshots: true + build_args: 'c("--no-manual","--compact-vignettes=gs+qpdf")' + error-on: '"error"' From fe6202380ca6d3fc33d8601a1b20299b0528e9a0 Mon Sep 17 00:00:00 2001 From: Hugo Gruson Date: Thu, 7 May 2026 15:13:01 +0200 Subject: [PATCH 2/4] Edit workflow name --- .github/workflows/R-CMD-check-SpatialDataPlot.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml index bca3a906..5eae2bdc 100644 --- a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml +++ b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml @@ -5,7 +5,7 @@ on: branches: [main, devel] pull_request: -name: R-CMD-check.yaml +name: check-SpatialData.Plot permissions: read-all From 69adefabe6ee59b39ae9931d088dc5877ec5fde3 Mon Sep 17 00:00:00 2001 From: Hugo Gruson Date: Thu, 7 May 2026 15:57:43 +0200 Subject: [PATCH 3/4] Try installing SpatialData from commit that triggered the workflow --- .github/workflows/R-CMD-check-SpatialDataPlot.yaml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml index 5eae2bdc..6576e619 100644 --- a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml +++ b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml @@ -42,7 +42,10 @@ jobs: - uses: r-lib/actions/setup-r-dependencies@v2 with: - extra-packages: any::rcmdcheck + # Install SpatialData from the commit that triggered the workflow + extra-packages: | + any::rcmdcheck + HelenaLC/SpatialData@${{ github.sha }} needs: check - uses: r-lib/actions/check-r-package@v2 From bc8c0eb8fcfa92ec16027dde304ec48b20dbb317 Mon Sep 17 00:00:00 2001 From: Hugo Gruson Date: Fri, 8 May 2026 13:16:13 +0200 Subject: [PATCH 4/4] Try patching SpatialData.plot DESCRIPTION on the fly --- .github/workflows/R-CMD-check-SpatialDataPlot.yaml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml index 6576e619..7468bfd0 100644 --- a/.github/workflows/R-CMD-check-SpatialDataPlot.yaml +++ b/.github/workflows/R-CMD-check-SpatialDataPlot.yaml @@ -40,6 +40,14 @@ jobs: r-version: ${{ matrix.config.r }} http-user-agent: ${{ matrix.config.http-user-agent }} + # This step can be removed once SpatialData is on Bioconductor, and is removed from Remotes: in SpatialData.plot's DESCRIPTION. + - name: Avoid SpatialData version conflict + run: | + desc <- read.dcf("DESCRIPTION") + desc[, "Remotes"] <- sub("HelenaLC/SpatialData,\\s*", "", desc[, "Remotes"]) + write.dcf(desc, "DESCRIPTION") + shell: Rscript {0} + - uses: r-lib/actions/setup-r-dependencies@v2 with: # Install SpatialData from the commit that triggered the workflow