Skip to content

Allow external libneo_ref trigger in CI#367

Merged
krystophny merged 1 commit into
mainfrom
ci/libneo-dispatch
Jun 8, 2026
Merged

Allow external libneo_ref trigger in CI#367
krystophny merged 1 commit into
mainfrom
ci/libneo-dispatch

Conversation

@krystophny

@krystophny krystophny commented Jun 8, 2026

Copy link
Copy Markdown
Member

Lets libneo's release gate build SIMPLE against a candidate ref. Standalone: SIMPLE's resolver already reads LIBNEO_BRANCH on main.

main.yml takes a libneo_ref dispatch input. LIBNEO_BRANCH now resolves to inputs.libneo_ref || github.head_ref || github.ref_name: a dispatched ref wins, otherwise the existing follow-branch default holds. The full suite runs on dispatch.

A libneo release dispatches main.yml with a candidate ref; it takes
precedence over the branch-name default. SIMPLE's resolver reads
LIBNEO_BRANCH.
@krystophny krystophny force-pushed the ci/libneo-dispatch branch from a240cd7 to e4383a5 Compare June 8, 2026 16:51
@krystophny krystophny enabled auto-merge (squash) June 8, 2026 16:54
@krystophny krystophny merged commit f649e9e into main Jun 8, 2026
7 checks passed
@krystophny krystophny deleted the ci/libneo-dispatch branch June 8, 2026 17:31
krystophny added a commit that referenced this pull request Jun 8, 2026
Gives the libneo reverse-dependency gate a sub-minute SIMPLE test set.

- Labels six sub-second tests `smoke`: three VMEC-coordinate tests that
call libneo directly (`test_magfie_refcoords_vmec`,
`test_jacobian_conventions_vmec`, `test_hctr_from_metric_vmec`) so the
gate actually catches a libneo break, plus the
`lapack`/`orbit_symplectic`/`field` base tests. Measured ~1.8s total.
- Adds a `test-smoke` make target (`ctest -L smoke`).
- `main.yml`: a libneo-gate dispatch (`workflow_dispatch` without
`full`) builds SIMPLE against the candidate libneo and runs only the
smoke set. Push, PR and `full=true` keep the complete fast + slow +
regression suite, unchanged.

Builds on #367 (already merged), which wired the `libneo_ref` dispatch
input and `LIBNEO_BRANCH` resolver.
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