ci: add workspace test gate (fmt, clippy, cargo test)#19
Open
BunsDev wants to merge 1 commit into
Open
Conversation
Every PR and push to main now runs the full workspace suite — including the RFC-0001 §5 conformance tests and C7 round-trip tests — plus rustfmt and clippy with warnings denied. Closes the observed gap: a fail-closed enforcement layer had no automated test gate on its own PRs.
There was a problem hiding this comment.
Pull request overview
Adds a baseline GitHub Actions CI workflow to enforce Rust workspace quality gates on every PR and on pushes to main, aligning the repo’s “fail-closed enforcement” goals with automated checks.
Changes:
- Introduces a new
CIworkflow triggered onpull_requestandpushtomain. - Runs
cargo fmt --all --check,cargo clippy --workspace --all-targets -- -D warnings, andcargo test --workspaceonubuntu-latest.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+23
to
+26
| - name: Clippy | ||
| run: cargo clippy --workspace --all-targets -- -D warnings | ||
| - name: Tests | ||
| run: cargo test --workspace |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the missing automated test gate. Every PR and push to
mainnow runs:cargo fmt --all --checkcargo clippy --workspace --all-targets -- -D warningscargo test --workspace— the full 98+ test suite, including the RFC-0001 §5 conformance tests and C7 round-trip testsWhy
Observed during the 2026-07-21 state review: all five open Phase 5 PRs show zero status checks — the repo had no CI at all. For a layer whose identity is fail-closed enforcement, PRs merging with no automated gate is the same failure shape the layer exists to prevent. This closes it with the minimal standard workflow (checkout@v4, dtolnay/rust-toolchain@stable, Swatinem/rust-cache@v2, read-only permissions).
Verification
cargo test --workspaceran green locally onmain(6476a67) this morning before branching; this workflow adds no code changes.Opened by Echo 🪞 under Val's "proceed with what's best" direction, 2026-07-21. Review + merge is Val/Nova's call — merging this first gives every Phase 5 PR a real check before the stack lands.