feat(R-CMD-check): per-leg extra-env via extra-config - #27
Merged
Merged
Conversation
The workflow-level extra-env from #26 is static, so it cannot express env that varies across the matrix. Require gates its slow tests on exactly one leg (R_REQUIRE_RUN_LONG_CI, driven by a runLong flag) specifically so its 11-job matrix does not camp on GHA's concurrency cap. That made Require the one repo that could not migrate onto this workflow at all. An extra-config leg may now carry its own `extra-env`, applied after the workflow-level block so a leg can override it. Generic rather than Require-shaped: any caller can vary env per leg. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012DVjmY3im9Xak7tXLSMGCa
This was referenced Aug 31, 2026
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
1 participant
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.
Last blocker for migrating every remaining package onto this workflow.
Why
extra-envfrom #26 is static, so it cannot express env that varies across the matrix.Requiregates its slow tests on exactly one leg —R_REQUIRE_RUN_LONG_CI, driven by arunLongflag — specifically so its 11-job matrix "doesn't camp on GHA's 20-job concurrency cap" (its own comment). That madeRequirethe one repo in the org that could not migrate at all.What
An
extra-configleg may now carry its ownextra-env:Applied after the workflow-level block, so a leg can override it. Generic rather than Require-shaped — any caller can vary env per leg.
Verified the parser locally for both the combined case and the common
MATRIX_ENVempty case.Context
With this, every one of the 17 repos still on a hand-rolled
R-CMD-check.yamlis expressible as a thin caller. Migration PRs follow.🤖 Generated with Claude Code
https://claude.ai/code/session_012DVjmY3im9Xak7tXLSMGCa