From: Duy Nguyen <pclouds@gmail.com>
Date: Sat, 17 Mar 2018 08:08:35 GMT
Subject: Re: [PATCH] format-patch: use --compact-summary instead of --summary
Message-ID: <CACsJy8DBDqin3xYk70A2HxrZ6AYJhd0V5dj98Ni+j8t0vdNGyQ@mail.gmail.com>
In-Reply-To: <xmqqlgerzojl.fsf@gitster-ct.c.googlers.com>

On Fri, Mar 16, 2018 at 10:41 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Nguyễn Thái Ngọc Duy  <pclouds@gmail.com> writes:
>
>> For reviewers, a couple fewer lines from diffstat (either cover letter
>> or patches) is a good thing because there's less to read.
>
> I can certainly accept that there may be some reviewers who share
> that view, but me personally, I would rather have them outside the
> main diffstat section, so addition and removal of files that are
> rarer events do stand out.
>
> I guess people would not mind an option to use it, but they can
> already say "git format-patch --compact-summary $upstream..", so...

Good point. I can't negate --summary though. But that calls for another patch.
-- 
Duy