Skip to content

ci: isolate native ABI smoke testing - #35

Merged
bernalde merged 1 commit into
mainfrom
codex/issue-34-streamline-ci
Jul 21, 2026
Merged

ci: isolate native ABI smoke testing#35
bernalde merged 1 commit into
mainfrom
codex/issue-34-streamline-ci

Conversation

@bernalde

@bernalde bernalde commented Jul 21, 2026

Copy link
Copy Markdown
Member

Summary

  • move the native C ABI build and two C-client executions into one dedicated Ubuntu job
  • keep the pinned upstream MQLib revision unchanged
  • disable unused coverage instrumentation in all three Julia compatibility jobs

Validation

  • actionlint
  • bash test/native_c_api_smoke.sh
    • built upstream commit 5854962
    • executed solve_qubo and c_api_smoke successfully
  • git -c core.whitespace=blank-at-eol,blank-at-eof,space-before-tab,cr-at-eol diff --check
  • full CI matrix (post-push; timings to be recorded from Actions)

Measurement

The #34 baseline ran the identical 25-second native smoke on both Ubuntu matrix jobs. This PR runs it once in parallel without Julia setup.

  • attempt 1: 10.4 aggregate runner-minutes; dedicated native smoke 30s
  • warm rerun: 8.3 aggregate runner-minutes; dedicated native smoke 32s
  • all Julia jobs logged coverage=false; the native ABI test executed exactly once per workflow run

The warm Windows job varied upward (3m45s to 4m45s), so the aggregate figures should not be treated as a deterministic benchmark. The structural saving is one fewer upstream clone/build and two fewer native client executions per workflow run.

Branch Hygiene

  • Base: main at 5664fcd2b943343ee9fad2aa7fd93d0b1743b468
  • Not stacked; open PR Make compatibility checks Dependabot-safe #33 changes test/runtests.jl, while this PR changes only the CI workflow.
  • Existing mixed line endings in ci.yml were preserved; whitespace validation treats CR at EOL correctly.

Closes #34

@bernalde
bernalde marked this pull request as ready for review July 21, 2026 08:53
@bernalde
bernalde merged commit 7a56bc4 into main Jul 21, 2026
8 checks passed
@bernalde
bernalde deleted the codex/issue-34-streamline-ci branch July 21, 2026 15:48
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.

Deduplicate native ABI smoke testing and disable unused CI coverage

1 participant