On 10/04/2024 15:22, Junio C Hamano wrote:
phillip.wood123@xxxxxxxxx writes:
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"?
Exactly
Thanks.