Skip to content

build: add test-ci target and use it in CI#23

Merged
cboone merged 1 commit intomainfrom
chore/ensure-tests-run
Feb 15, 2026
Merged

build: add test-ci target and use it in CI#23
cboone merged 1 commit intomainfrom
chore/ensure-tests-run

Conversation

@cboone
Copy link
Copy Markdown
Owner

@cboone cboone commented Feb 15, 2026

Summary

  • make build now runs unit tests before compiling by splitting compile-only work into a new binary target
  • added a dedicated test-ci target that runs the deterministic CI suite via test-all
  • updated the CI workflow to call make test-ci so test orchestration is defined in one place

Testing

  • make test-ci

Copilot AI review requested due to automatic review settings February 15, 2026 00:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reorganizes the build and test workflow to better centralize test orchestration in the Makefile. The build target now runs unit tests before compiling (with a new binary target for compile-only work), and a dedicated test-ci target consolidates the CI test suite definition in one place instead of having it split between the Makefile and CI workflow.

Changes:

  • Refactored build target to run unit tests before compilation, with binary target for compile-only operations
  • Added test-ci target that orchestrates the complete CI test suite via test-all
  • Updated CI workflow to delegate all test orchestration to make test-ci

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Makefile Split build into test+binary, added test-ci target, updated dependencies for test-cli/test-cli-live to use binary
.github/workflows/ci.yml Consolidated separate unit and CLI test steps into single test-ci invocation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cboone cboone merged commit bc30405 into main Feb 15, 2026
7 checks passed
@cboone cboone deleted the chore/ensure-tests-run branch February 15, 2026 00:55
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