Skip to content

Commit d32801b

Browse files
committed
fix: Add high-level guidance
1 parent e73e6b9 commit d32801b

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

text/3389-manifest-lint.md

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,17 @@ This does not allow sharing lints across workspaces.
196196
# Rationale and alternatives
197197
[rationale-and-alternatives]: #rationale-and-alternatives
198198

199-
This could be left to `clippy.toml` but that leaves `rustc` without a solution.
199+
When designing this, we wanted to keep in mind how things work today, including
200+
- `clippy` defines all configuration as linter/tool config and not lint config (linter/lint config is a future possibility)
201+
- All `clippy` lints are disjoint
202+
- `rustdoc` has no plans for groups outside of `all`
203+
- `rustc` today has some intersecting groups
204+
205+
However, we also need to consider how decisions might limit us in the future and whether we want to bind future decisions with this RFC, including
206+
- Whether existing decisions will be revisited
207+
- When new tools are added, like `cargo` and `cargo-semver-check`, which haven't had lint levels and configuration long enough (or at all) to explore their problem and design space.
208+
209+
This could be left to `clippy.toml` but that leaves `rustc`, `rustdoc`, and future linters without a solution.
200210

201211
`[lints]` could be `[package.lints]`, tying it to the package unlike `[patch]`
202212
and other fields that are more workspace related. Instead, we used

0 commit comments

Comments
 (0)