From: Junio C Hamano Date: Wed, 10 Apr 2024 14:22:06 GMT Subject: Re: [PATCH 3/3] t3428: restore coverage for "apply" backend Message-ID: In-Reply-To: <6ab115bf-7b78-4633-a64c-9a0925d68e3f@gmail.com> phillip.wood123@gmail.com 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"? Thanks.