Subject: Re: [PATCH v3] doc: Add some guidelines for reviewing.

From: Maxim Cournoyer <[email-masked]>
Date: Tue, 07 Nov 2023 11:14:00 -0500

Hello,

Ludovic Courtès <[email-masked]> writes:

> Hello,
>
> Maxim Cournoyer <[email-masked]> skribis:
>
>> * doc/contributing.texi (Contributing) [Reviewing the Work of Others]: New
>> section.
>> (Debbugs Usertags): Expound with Emacs Debbugs information and document the
>> 'reviewed-looks-good' usertag.
>>
>> Change-Id: I56630b15ec4fbc5c67e5420dbf2838556a005d6b
>> ---
>>
>> Changes in v3:
>>  - Replace LGTM with Reviewed-by Git tag
>>  - Add b4 config
>>  - Link to the Submitting Patches section for check list
>>  - Fuse further suggestions by both Ludovic and Simon
>
> Could you mention the b4 change in the commit log?  Otherwise LGTM!

Done, thanks for the review.

>> +[b4]
>> +        attestation-check-dkim = off
>> +        attestation-policy = off
>> +        linkmask = https://yhetil.org/guix/%s
>> +        linktrailermask = https://yhetil.org/guix/%s
>> +        midmask = https://yhetil.org/guix/%s
>
> Really cool to have the b4 workflow documented and a default config in
> place!

Now we could document more extensively some successful workflows in the
Cookbook (Debbugs/Gnus-based, QA-based, b4-based, etc.) :-).

-- 
Thanks,
Maxim