Repository navigation
Replace Deno CI with pure Rust validation workflow - #9
tj (tjkilgore93-stack) with Copilot wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: tjkilgore93-stack <313495784+tjkilgore93-stack@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
The Rust workflow will fail because the repository contains no Cargo manifest or Rust crate.
1 open finding
What changed in this PR
Replaces the Deno workflow with Rust-focused CI validation.
Changes:
- Removes Deno linting and testing.
- Adds Rust formatting, compilation, and test checks.
| File | Description |
|---|---|
.github/workflows/rust.yml |
Adds Rust CI validation. |
.github/workflows/deno.yml |
Removes legacy Deno CI. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| - name: Check formatting | ||
| run: cargo fmt --all --check | ||
|
|
||
| - name: Run cargo check | ||
| run: cargo check --verbose |
| - name: Check formatting | ||
| run: cargo fmt --all --check |
There was a problem hiding this comment.
🔴 Validation fails without a Cargo project
Every run fails at cargo fmt --all --check because the repository has no Cargo manifest. Neither cargo check nor cargo test runs.
Learn more
The workflow runs at the repository root after checkout. This repository contains package.json and JavaScript source, but no Cargo.toml or Rust source. Cargo needs a manifest before it can format, check, or test a project, so every run stops at the first Cargo command.
Example: A pull request changes src/core/billing.js. Checkout succeeds, then cargo fmt --all --check reports that no Cargo.toml exists; the job fails without validating the change.
Recommended fix: Replace the Cargo steps with checks for the actual Node project, using the Node version specified by package.json and installing dependencies before running its checks. If Rust validation is intended for a future project, introduce its manifest and sources first.
Was this helpful? React with 👍 or 👎 to provide feedback.

The legacy Deno CI workflow was removed to stop irrelevant lint and test failures across the repository. This change introduces a minimal, standard GitHub Actions workflow dedicated purely to validating Rust tooling.
Workflow Changes
.github/workflows/deno.yml..github/workflows/rust.ymlconfigured to:pushandpull_requestagainstmain.contents: read).rustfmtcomponent viadtolnay/rust-toolchain.cargo fmt --all --check,cargo check --verbose, andcargo test --verbose.Workflow Definition