Skip to content

Commit 5092e4f

Browse files
cpsievertclaude
andcommitted
Fix GGally exclusion for R 4.2 (oldrel-3)
Use matrix variable with fallback instead of ternary expression that was passing literal 'NULL' string to pak. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent b47c33e commit 5092e4f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/R-CMD-check.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ jobs:
3434
- {os: ubuntu-latest, r: 'release'}
3535
- {os: ubuntu-latest, r: 'oldrel-1'}
3636
- {os: ubuntu-latest, r: 'oldrel-2'}
37-
- {os: ubuntu-latest, r: 'oldrel-3'}
37+
# GGally requires R >= 4.3, so exclude it on oldrel-3 (R 4.2)
38+
- {os: ubuntu-latest, r: 'oldrel-3', packages: 'any::sessioninfo, !GGally'}
3839
# - {os: ubuntu-latest, r: 'oldrel-4'} # dependency issues with oldrel-4
3940

4041
env:
@@ -62,8 +63,7 @@ jobs:
6263
with:
6364
cache-version: 3
6465
needs: check
65-
# GGally doesn't install on R 4.2 (oldrel-3); tests skip via skip_if_not_installed()
66-
packages: ${{ matrix.config.r == 'oldrel-3' && '"deps::., !GGally"' || 'NULL' }}
66+
packages: ${{ matrix.config.packages || 'any::sessioninfo' }}
6767

6868
- name: Set up Python (for reticulate)
6969
if: matrix.config.visual_tests == true

0 commit comments

Comments
 (0)