From 8048d67e5917b771be9aa819094aebf3936d0e98 Mon Sep 17 00:00:00 2001 From: main Date: Sat, 4 Apr 2026 21:50:22 -0400 Subject: docs: manifest-own warning fatality --- docs/rust-linting-proposal.md | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'docs/rust-linting-proposal.md') diff --git a/docs/rust-linting-proposal.md b/docs/rust-linting-proposal.md index 53b9ae2..88e5612 100644 --- a/docs/rust-linting-proposal.md +++ b/docs/rust-linting-proposal.md @@ -22,6 +22,10 @@ One extra local fact matters: there is effectively no checked-in Rust CI posture One more distinction matters: Clippy’s `too_many_lines` is about function bodies, not whole source files. If the goal is to force large modules to split, the starter pack needs an explicit file-level check instead of pretending the existing lint already covers it. +One more ratchet should be explicit: if the repo wants warnings to be fatal, that belongs in +`[workspace.lints.rust]` as `warnings = "deny"`, not in runner-side `cargo clippy -- -D warnings` +tails. The manifest owns policy; the runner merely executes it. + ## Local Survey ### The old shape @@ -194,6 +198,7 @@ rust-version = "1.xx" version = "0.1.0" [workspace.lints.rust] +warnings = "deny" elided_lifetimes_in_paths = "deny" unexpected_cfgs = "deny" unsafe_code = "deny" -- cgit v1.2.3