Skip to content

feat: add linear GitHub Actions CI - #4

Merged
zatevakhin merged 1 commit into
v2/agent-controlfrom
v3/ci
Sep 16, 2026
Merged

zatevakhin merged 1 commit into
v2/agent-controlfrom
v3/ci

Conversation

@zatevakhin

@zatevakhin zatevakhin commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add a single linear CI graph: format → clippy → test → build → nix.
  • Reuse qmtui toolchain/cache/Nix installer actions and vllm.cpp-rs concurrency/timeouts, without parallel fan-in or cross-compile/release jobs.
  • Pin flake.lock so the nix job can run alejandra --check and nix develop --command cargo test.
  • Build the release binary and smoke qmtacp --help. Live ACP/server tests stay out of CI.

Stacked on #3 (v2/agent-control).

@coderabbitai ignore

Why

qmtacp had no CI. PRs currently rely on local cargo test / clippy, so format, lint, tests, and the flake shell can drift independently. A linear needs chain fails fast on style, then lints, then unit tests, then a real binary, then Nix, without diamonds or extra workflows.

Test plan

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test --all-targets --all-features (26 passed)
  • cargo build --release --bin qmtacp and ./target/release/qmtacp --help
  • alejandra --check flake.nix
  • GitHub Actions run on this PR

@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing.

Comment @coderabbitai help to get the list of available commands.

@zatevakhin
zatevakhin added this pull request to stack #6 September 16, 2026 13:31
@zatevakhin
zatevakhin marked this pull request as ready for review September 16, 2026 14:42
Run format, clippy, tests, release-bin smoke, and nix develop tests
in a single needs chain. Pin flake.lock so the nix job is reproducible.
@zatevakhin
zatevakhin merged commit cfdf072 into main Sep 16, 2026
6 checks passed
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