From: Jeff King <peff@peff.net>
Date: Tue, 11 Oct 2022 00:59:08 GMT
Subject: Re: [RFC PATCH v2] shortlog: add group-by options for year and month
Message-ID: <Y0S/3Nl6PFWS3rTd@coredump.intra.peff.net>
In-Reply-To: <Yz93RjrJ00A5QvNe.jacob@initialcommit.io>

On Thu, Oct 06, 2022 at 05:48:06PM -0700, Jacob Stopak wrote:

> > We probably want to insist that the format contains a "%" sign, and/or
> > git it a keyword like "format:". Otherwise a typo like:
> > 
> >   git shortlog --format=autor
> > 
> > stops being an error we detect, and just returns nonsense results
> > (every commit has the same ident).
> 
> Small aside: I like how Taylor re-used the --group option for the custom format.
> IMO it hammers home that this is a grouping option and not just formatting or
> filtering which can be confusing to users sometimes when doing data analytics.

Yeah, sorry this was just a typo/thinko on my part. It absolutely should
be --group, as --format already does something else.

-Peff