Re: [PATCH 01/22] rust: init: disable doctests

From: Miguel Ojeda
Date: Wed Mar 05 2025 - 08:01:09 EST


On Wed, Mar 5, 2025 at 1:53 PM Benno Lossin <benno.lossin@xxxxxxxxx> wrote:
>
> Thanks for pointing that out, any suggestions for this concrete case? I
> wasn't able to come up with an imperative statement.

Normally I follow a "Context, Why, (thus) What, Alternatives, Future"
format or similar.

The imperative is meant for the "What" part, i.e. the actual change
you are performing, which you already do in "..., ignore all doctests
...", so that looks fine to me.

You could just remove the first line, or perhaps move it and reword it
with ", in order to..." after the "ignore" sentence.

Cheers,
Miguel