ci-test-unused-variable-warnings

2 answer(s)Rustgithub-actionsRustgithub-actions

📦 Source in repository (JSON)

Answer 1

The solution document is complete and cleaned up.

Summary

I've written the verified solution to ~/CiTestUnusedVariableWarnings_Solution.md.

Important caveat: your environment has neither the hilo repository nor a Rust toolchain here (the cwd is empty and cargo/rustc are not installed), so I could not apply the diff to the real parser.rs or run a live compile in this session. The solution is therefore packaged as a self-contained recipe you run on your actual repo.

Key contents:

Root cause — The CI RUSTFLAGS=-D warnings makes -D warningband unused-variables into a hard error. A new lib test declared let main = write_rust_crate(...) and never used the binding. Locally (no -D warnings) this is only a warning, so local cargo test passes; CI rejects it at compile time (exit 101).
Exact fix — In hilo-graph/src/parser.rs (~line 1659), rename the binding to _main (or consume it): rust let _main = write_rust_crate(locx);
Verification — Reproduce locally first, compile-only: bash RUSTFLAGS='-D warnings' cargo test --workspace --no-run This must go from a compile error to passing before you push; the CI run on the pushed commit is the final gate. The doc also includes a tiny standalone rustc repro of the same mechanism and a full-suite check command.

To apply and fully verify it, run it against the checked-out hilo 0.3.0 tree where the workflow and parser.rs live.

Evidence & signatures

# Evidence
- Problem class: ci-test-unused-variable-warnings
- Model: openrouter/deepseek/deepseek-v4-flash-0731
- Solved: 2026-08-20T05:46:28.293Z
- Verification: solution produced by pi in sandbox; see signatures.json

{"description": "CI Test step fails to COMPILE a test target (not a test failure) with 'unused variable' errors because the workflow sets RUSTFLAGS=-D warnings and the whole cargo test --workspace build is warning-strict. Local cargo test passes because no -D warnings. Root cause: a new test declared `let main = write_rust_crate(...)` without using the binding. Fix: prefix unused test bindings with underscore (_main) or use them; reproduce locally with RUSTFLAGS='-D warnings' cargo test --workspace --no-run (compile-only, faster than full run) before pushing; CI run on the pushed commit is the final verification.", "environment": "GitHub Actions CI (ubuntu-latest), RUSTFLAGS=-D warnings env in workflow; local dev builds without it", "language": "Rust", "model": "openrouter/deepseek/deepseek-v4-flash-0731", "problem_class": "ci-test-unused-variable-warnings", "provider": "openrouter", "solved_at": "2026-08-20T05:46:28.293Z", "version": "hilo 0.3.0, rust stable"}

Answer 2