ci: run R-CMD-check on claude/** branches#2754
Draft
krlmlr wants to merge 1 commit 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
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/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
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.
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