Skip to content

ci: Sync R-CMD-check workflow; drop matrix-env plumbing from custom action - #2422

Merged
krlmlr merged 1 commit into
mainfrom
claude/gha-env-var-setup-ej3a28
Jul 27, 2026
Merged

ci: Sync R-CMD-check workflow; drop matrix-env plumbing from custom action#2422
krlmlr merged 1 commit into
mainfrom
claude/gha-env-var-setup-ej3a28

Conversation

@krlmlr

@krlmlr krlmlr commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Sync .github/workflows/R-CMD-check.yaml and
.github/workflows/versions-matrix/action.R from cynkratemplate
(cynkra/cynkratemplate#95).

The template's rcc-full job now applies the generic env field
of test matrix entries to $GITHUB_ENV itself,
before the custom before-install action runs,
because composite actions cannot read the matrix context —
until now, every consumer repo had to carry
the same apply-to-$GITHUB_ENV boilerplate in its custom action,
and a repo without it silently dropped the variables.

The apply boilerplate and the env input
are therefore removed from the custom before-install action;
DUCKDB_R_POISON_ENGINE and DUCKDB_R_USE_SYSTEM_LIB
from the engine-poisoning and vendored-build matrix entries
keep reaching the tripwire and system-libduckdb steps
through the workflow-level step,
which runs before the custom action.

The same branch is also pushed to krlmlr/duckdb-r
(the vendoring pipeline repo, which shares these workflow files);
that copy is krlmlr/duckdb-r@claude/gha-env-var-setup-ej3a28.

🤖 Generated with Claude Code

https://claude.ai/code/session_0136C9bRMjBGkUrtLdkYNmAg


Generated by Claude Code

…ction

Sync .github/workflows/R-CMD-check.yaml and
.github/workflows/versions-matrix/action.R from cynkratemplate
(cynkra/cynkratemplate#95).
The rcc-full job now applies the generic "env" field
of test matrix entries to $GITHUB_ENV itself,
before the custom before-install action runs,
because composite actions cannot read the matrix context.

The apply-to-GITHUB_ENV boilerplate and the "env" input
in the custom before-install action are therefore no longer needed;
DUCKDB_R_POISON_ENGINE and DUCKDB_R_USE_SYSTEM_LIB
from the engine-poisoning and vendored-build matrix entries
keep reaching the tripwire and system-libduckdb steps
through the workflow-level step.
@krlmlr
krlmlr enabled auto-merge (squash) July 27, 2026 18:44
@krlmlr
krlmlr merged commit 620bcb6 into main Jul 27, 2026
5 of 6 checks passed
@krlmlr
krlmlr deleted the claude/gha-env-var-setup-ej3a28 branch July 27, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants