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

Nikolai Kondrashov
2023-11-16T12:14:33Z

On 11/15/23 22:38, Konstantin Ryabitsev wrote:
> On Wed, Nov 15, 2023 at 07:43:49PM +0200, Nikolai Kondrashov wrote:
>> Introduce a new tag, 'Tested-with:', documented in the
>> Documentation/process/submitting-patches.rst file. The tag is expected
>> to reference the documented test suites, similarly to the 'V:' field,
>> and to certify that the submitter executed the test suite on the change,
>> and that it passed.
>
> I'm not sure it should be a tag that stays all the way through git commit.
> How about making it a cover letter/first patch footer?
>
> tested-with: <suite>

Yes, that would be better indeed. However, checkpatch.pl doesn't process cover
letters, and so we would have no automated way to advertise and nudge people
towards testing.

Nick

P.S. Git forges do that for you by nature of actually running the tests
themselves, automatically. *Ducks*