Re: [PATCH] rust: allow `clippy::disallowed_names` for doctests
From: John Hubbard
Date: Mon Nov 17 2025 - 16:38:10 EST
On 11/17/25 1:26 PM, Miguel Ojeda wrote:
> On Mon, Nov 17, 2025 at 10:13 PM John Hubbard <jhubbard@xxxxxxxxxx> wrote:
>>
>> The current set of disallowed names is inherited from clippy's defaults [1],
>> which are "foo", "baz", and "quux". This may be extended via .clipy.toml,
>> which so far has no entries for disallowed names.
>
> As for this paragraph, I am confused why we would want to add it.
>
> This isn't about extending the disallowed names or about how the lint
> works in general. It is just about disabling the lint entirely for
> doctests.
Yes. But I still thought some context might help, because it wasn't
obvious at first to me (as a Rust learner).
>
> So adding that paragraph would make it the commit more confusing.
>
OK, I'll accept that judgment call. :)
thanks,
--
John Hubbard