Re: [PATCH 3/3] t3428: restore coverage for "apply" backend

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

 



On 12/04/2024 10:33, phillip.wood123@xxxxxxxxx wrote:
On 10/04/2024 17:45, Junio C Hamano wrote:
phillip.wood123@xxxxxxxxx writes:
Perhaps we can write that in the log message to help the next person
who reads the patch?  Something like...

     t3428: restore coverage for "apply" backend
     This test file assumes the "apply" backend is the default which is
     not the case since 2ac0d6273f (rebase: change the default backend
     from "am" to "merge", 2020-02-15).  The way "merge" backend honors
     "--signoff" is already tested elsewhere, so make sure the "apply"
     backend is tested by specifying it explicitly.
     Signed-off-by: Phillip Wood <phillip.wood@xxxxxxxxxxxxx>
     Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>

Sounds good, I'll send a re-roll

Oh, it looks like this hit next yesterday

Phillip




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux