ci: run R-CMD-check on claude/** branches - #2754
Draft
krlmlr wants to merge 2 commits into
Draft
Conversation
krlmlr
force-pushed
the
claude/ci-full-matrix
branch
from
July 24, 2026 10:22
64a71f6 to
87f4023
Compare
krlmlr
force-pushed
the
claude/fix-names-deprecation-graph
branch
from
July 24, 2026 10:52
1d82032 to
6566387
Compare
krlmlr
force-pushed
the
claude/ci-full-matrix
branch
2 times, most recently
from
July 24, 2026 18:36
3b874e2 to
1c0402e
Compare
krlmlr
changed the title
claude/** branches
1 task
krlmlr
force-pushed
the
claude/fix-names-deprecation-graph
branch
from
July 24, 2026 18:38
6566387 to
aa70783
Compare
krlmlr
force-pushed
the
claude/ci-full-matrix
branch
from
July 24, 2026 18:38
1c0402e to
4a890a6
Compare
1 task
krlmlr
force-pushed
the
claude/ci-full-matrix
branch
from
July 26, 2026 10:35
4a890a6 to
d21c3f0
Compare
'graph' is an optional Enhances dependency published only on Bioconductor. The Bioconductor releases matching R 4.2 / 4.3 (graph 3.16 / 3.18) are archived and return HTTP 404, so pak dependency resolution aborts on those jobs before anything is built. Use pak's documented downstream-package parameter instead of touching the workflows: Config/gha/extra-packages: graph=?ignore-before-r=4.4.0 The install action already forwards Config/gha/extra-packages to setup-r-dependencies, and pak parses graph=?ignore-before-r=4.4.0 as a 'param' reference that applies to the graph dependency pulled in via deps::., ignoring it on R older than 4.4.0 and keeping it on 4.4 and newer. This replaces an earlier attempt that set PKG_USE_BIOCONDUCTOR=false from a workflow step. That approach did not work: disabling Bioconductor only turned the 404 into "Can't find package called graph", because pak does not drop an unavailable soft dependency unless it is explicitly ignored. It also broke the Windows jobs, which could not run the multi-line Rscript -e invocation. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019E6CUgb5aFw6uvWXztcHgV
krlmlr
force-pushed
the
claude/fix-names-deprecation-graph
branch
from
July 26, 2026 16:12
9d1ec13 to
62036bc
Compare
The workflow only triggered on push to main/master/release/next/cran-* and on pull_request against main/master, so a PR based on another claude/** branch got no CI runs at all. Add claude/** to both branch filters. The push trigger is the one that matters for coverage: the versions-matrix step is skipped for same-repo pull requests, but runs for push events, so pushing a claude/** branch generates the version matrix and exercises the rcc-full job. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019E6CUgb5aFw6uvWXztcHgV
krlmlr
force-pushed
the
claude/ci-full-matrix
branch
from
July 26, 2026 16:12
d21c3f0 to
2fc5dfa
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This can be reused to check the full matrix which is omitted by default for maintainers (=PRs from the igraph/rigraph repo). Retarget to the appropriate claude/** branch.
LLM disclosure: this PR was prepared with Claude Code (Claude Opus 4.8).
What (draft)
R-CMD-check.yamlonly triggered onpushtomain/master/release/next/cran-*and onpull_requestagainstmain/master. A PR based on anotherclaude/**branch (like this one, stacked on #2753) therefore got no CI runs at all.This adds
claude/**to both branch filters — a two-line change:The
pushtrigger is the one that matters for coverage: theversions-matrixstep is skipped for same-repo pull requests (itsif:requires a fork PR or acran-*branch), but it does run forpushevents. So pushing aclaude/**branch generates the full version matrix and exercises thercc-fulljob.Not included
An earlier revision of this PR also widened the matrix in
.github/workflows/versions-matrix/action.R(full R range on macOS/Windows/arm64). That was a misunderstanding of "test the full matrix" and has been reverted — the matrix definition is unchanged, including its deliberately ragged arm64/macOS coverage and the rationale comments.Stacking
Based on #2753 (
claude/fix-names-deprecation-graph), notmain. Review/merge #2753 first; this then reduces to the two-line trigger diff.Note
The
claude/**trigger runs CI on every push to anyclaude/*branch. That's what makes these stacked PRs testable, but you may want to narrow or drop it before it lands onmain.🤖 Generated with Claude Code