Junio C Hamano <gitster@xxxxxxxxx> writes: > Sergey Organov <sorganov@xxxxxxxxx> writes: > >> Here is the patch you've asked for, separately, as I honestly believe it >> does not belong to the series in question themselves. This is on top of >> the series. > > Well, the tip commit's title being "add test for --diff-merges=off", > testing it standalone as well as in different ways to combine with > another option that potentially interacts with it would make the > commit complete, I would think. My point is that this set of commits is essentially "make --diff-merges=on a synonym for --no-diff-merges", so it should only include the same test(s) that exist for --no-diff-merges. If new tests are to be added, they should have been added before the series, or after them, with their own reasoning, I think. Yeah, not a big deal, just a bit of perfectionism here. Thanks, -- Sergey