guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#66436: [PATCH v3] doc: Add some guidelines for reviewing.


From: Maxim Cournoyer
Subject: bug#66436: [PATCH v3] doc: Add some guidelines for reviewing.
Date: Tue, 07 Nov 2023 11:14:00 -0500
User-agent: Gnus/5.13 (Gnus v5.13)

Hello,

Ludovic Courtès <ludo@gnu.org> writes:

> Hello,
>
> Maxim Cournoyer <maxim.cournoyer@gmail.com> 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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]