Skip to content

Pick the devel-vctrs push-time tests by a marker line - #181

Merged
jmgirard merged 7 commits into
mainfrom
m165-devel-vctrs-drift
Oct 31, 2026
Merged

jmgirard merged 7 commits into
mainfrom
m165-devel-vctrs-drift

Conversation

@jmgirard

Copy link
Copy Markdown
Collaborator

The devel-vctrs workflow named its three push-time test files by hand, so a renamed file dropped out with no signal, and its concurrency group repeated the test step's whole-suite switch with nothing keeping the two copies equal.

  • On a push or pull request, the test step now runs the tests/testthat/test-*.R files that carry the line # devel-vctrs.yaml runs this file on every push and pull request., and stops with an error when none does. The three vctrs compatibility files carry it.
  • The step decides the whole-suite switch from GITHUB_EVENT_NAME and inputs.full, and the concurrency group is keyed on those two raw values with the workflow and ref.
  • The weekly schedule and a full dispatch still run the whole suite. A scheduled run and a full dispatch now sit in different groups and can run side by side.

No package code change.

Candidate row: Guard the devel-vctrs filter from drift

@jmgirard
jmgirard merged commit bb968b2 into main Oct 31, 2026
17 checks passed
@jmgirard
jmgirard deleted the m165-devel-vctrs-drift branch October 31, 2026 13:54
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.

1 participant