On Sat, Jan 16, 2021 at 04:35:49PM +0100, Ævar Arnfjörð Bjarmason wrote: > From: Jeff King <peff@xxxxxxxx> > > Rewrite a brittle tests which used "rev-list" without "--[no-]merges" > to figure out if a set of commits turned into merge commits or not. > > Signed-off-by: Jeff King <peff@xxxxxxxx> > Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> > Commit-message-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> Thanks for picking this up. I think it's a cleanup worth doing. I didn't actually sign-off the original I sent, though. I'm quite happy to do so now, and in general everything I send to the list as a "something like this" patch will be done with the intent that it could be contributed under the DCO. But I think in such a case, we should stop short of forging the signoff. And ideally get an explicit signoff, or lacking that just add your own to assert DCO's paragraph (b). Anyway, this is most definitely: Signed-off-by: Jeff King <peff@xxxxxxxx> -Peff