doc-valid-idents not respected in Cargo.toml [lints] table

[rockboynton]

Summary

See title

Reproducer

I tried this code:

[lints.clippy]
pedantic    = "deny"
doc_markdown = { level = "deny", doc_valid_idents = ["GHz", ".."] }

//! GHz

I expected to see this happen:

A clean build

Instead, this happened:

I still see the error on the GHz in my doc comment

Version

❯ rustc -Vv
rustc 1.74.0-nightly (58eefc33a 2023-08-24)
binary: rustc
commit-hash: 58eefc33adf769a1abe12ad94b3e6811185b4ce5
commit-date: 2023-08-24
host: x86_64-unknown-linux-gnu
release: 1.74.0-nightly
LLVM version: 17.0.0

Additional Labels

No response

[bayareaunicorn]

Lints in Rust can be controlled and customized using the #[allow(lint_name)] attribute in the code. This attribute allows you to selectively disable specific lints for specific sections of code. While this attribute is typically used within the code itself, it may be worth exploring if it can be applied to the Cargo.toml file as well.

[rockboynton]

Hi @Stoned33, thanks for your reply. While that does work per crate, other lints work with overriding them in the Cargo.toml as in my example, although I do notice it only seems to work when it's only the bare level key.

For example, this works as expected, to allow denying the pedantic group with an exception, since the order matters in the file

[lints.clippy]
pedantic    = "deny"
borrow_as_ptr = "allow" 

[samueltardieu]

Isn't lint extra configuration supposed to be given in clippy.toml? If you create one with

doc-valid-idents = ["GHz", ".."]

it should do what you want.

[Alexendoo]

Opened a cargo issue for this: rust-lang/cargo#12917