Re: [PATCH v5 2/2] checkpatch: warn on known non-plural rust doc headers and empty doc comments

[Miguel Ojeda]

On Wed, Oct 2, 2024 at 4:29 AM Patrick Miller <paddymills@xxxxxxxxx> wrote:
>
> Signed-off-by: Patrick Miller <paddymills@xxxxxxxxx>

These tags are typically placed before your Signed-off-by (tags after
it are usually the ones added by the next person handling the patch,
i.e. the next Signed-off-by).

> Co-developed-by: Hridesh MG <hridesh699@xxxxxxxxx>

This tag requires a Signed-off-by -- please see:

    https://docs.kernel.org/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by

> Suggested-by: Trevor
> Gross <tmgross@xxxxxxxxx>

It seems like the patch has been wrapped in a couple places.

>   - merged Hridesh MG's patch[2] to check against consecutive rustdoc comments
>   - revised Hridesh MG's
>  patch to match against $prevrawline being new
>       or existing
>   - added fix to Hridesh MG's patch

It may be clearer and simpler (for attribution purposes) to avoid
merging them, and instead add a check in each patch (you can still
rebase Hridesh's into yours on top, so that you add it inside the
`realfile` condition etc.).

I would also include Hridesh's cleanup here too if we are doing
everything here. Something like:

#1: clean A
#2: add check for A
#3: clean B
#4: add check for B

> +# checks for rust files

We already have other checks for Rust files that are shared with other
languages. Perhaps we can be slightly more clear with "checks for Rust
files (only)" or "Rust-only checks" or "checks that only apply to Rust
files"?

Thanks!

Cheers,
Miguel