From: Martin Ågren <martin.agren@gmail.com>
Date: Sun, 27 Sep 2020 14:38:54 GMT
Subject: Re: [PATCH v2 0/8] parsing trailers with shortlog
Message-ID: <CAN0heSqXsnMq5TiMaDow4GKQ_uWnjb=a7JRvD854JocSK5Hrrg@mail.gmail.com>
In-Reply-To: <20200927083933.GA2222823@coredump.intra.peff.net>

On Sun, 27 Sep 2020 at 10:39, Jeff King <peff@peff.net> wrote:
>
> Here's a re-roll with some cosmetic fixups. All except the first address
> the points raised by Eric and Martin (thanks, both):
>
>   - put the --group=<type> unordered list into a delimited block, so
>     that the follow-on paragraph about multiple --group options isn't
>     indented with "trailer" item

Yep, that makes sense. The result looks good.

This v2 addresses my comments on v1 and, FWIW, looks good to me.

Martin