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

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 10, 2024, 14:22 UTC
Message-ID: <xmqqjzl5z4a9.fsf@gitster.g>
In-Reply-To: <6ab115bf-7b78-4633-a64c-9a0925d68e3f@gmail.com>

phillip.wood123@gmail.com writes:

Show 7 quoted lines

>> Hmph, doesn't this lose coverage for the merge backend, though?
>
> I don't think so, we had coverage for the merge backend from the other
> tests before 2ac0d6273f as all of the other tests in this file use the
> merge backend. We're no longer testing "--signoff" without specifying
> some other option that selects a backend but it seems unlikely that we
> could break that without breaking one of the other tests.

OK, so we have "rebase --merge --signoff" tested elsewhere and we are replacing "rebase --signoff" with "rebase --apply --signoff"?

Thanks.