diff options
Diffstat (limited to 'docs')
| -rw-r--r-- | docs/bootstrap-fresh.md | 1 | ||||
| -rw-r--r-- | docs/bootstrap-retrofit.md | 3 | ||||
| -rw-r--r-- | docs/rust-linting-proposal.md | 5 |
3 files changed, 9 insertions, 0 deletions
diff --git a/docs/bootstrap-fresh.md b/docs/bootstrap-fresh.md index f77b819..89bf757 100644 --- a/docs/bootstrap-fresh.md +++ b/docs/bootstrap-fresh.md @@ -116,6 +116,7 @@ The template only installs the fast gate. Add the deeper posture when the repo i ## Acceptance Checklist - root `Cargo.toml` contains `[workspace.lints.rust]`, `[workspace.lints.rustdoc]`, and `[workspace.lints.clippy]` +- root `Cargo.toml` owns warning-fatality policy, e.g. `warnings = "deny"` under `[workspace.lints.rust]`, rather than hiding it in runner flags - root `Cargo.toml` contains canonical command vectors in `[workspace.metadata.rust-starter]` - root `Cargo.toml` contains an ordered `canonicalize_commands` pipeline in `[workspace.metadata.rust-starter]` - root `Cargo.toml` contains `[workspace.metadata.rust-starter.source_files]` with an intentional `max_lines` value diff --git a/docs/bootstrap-retrofit.md b/docs/bootstrap-retrofit.md index c07be56..a2910ce 100644 --- a/docs/bootstrap-retrofit.md +++ b/docs/bootstrap-retrofit.md @@ -96,6 +96,9 @@ Add `[lints] workspace = true` to every member crate. Take inline `cargo clippy -- -A/-D ...` tails from scripts and CI, and migrate them into the root manifest in grouped, commented form. +That includes `-D warnings`: if warnings are meant to be fatal, encode that as +`warnings = "deny"` under `[workspace.lints.rust]` instead of leaving it as a command-line tail. + ### 5. Tighten local suppression discipline Prefer: 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" |