Re: [PATCH 1/3] MAINTAINERS: Introduce V: field for required tests

Mark Brown
2023-11-15T20:01:32Z

[-- Attachment #1: Type: text/plain, Size: 680 bytes --]

On Wed, Nov 15, 2023 at 10:31:21AM -0800, Joe Perches wrote:
> On Wed, 2023-11-15 at 19:43 +0200, Nikolai Kondrashov wrote:
> > Introduce a new 'V:' ("Verify") field to MAINTAINERS. The field accepts
> > a name of a test suite which is required to be executed for each
> > contribution to the subsystem.

> Perhaps this is simply too much overhead
> process requirements for most kernel work.

> While the addition of V: seems ok, I think
> putting the verification in checkpatch is
> odd at best and the verification of test
> execution should be a separate script.

Verifying that the expected tags are present and valid does seem firmly
in what checkpatch's wheelhouse though?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]