Re: [PATCH v3 0/7] shortlog: introduce `--group=<format>`

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 21, 2022 at 06:25:37PM -0400, Taylor Blau wrote:

> Here is a (likely final) reroll of my series to implement arbitrary
> pretty formats as shortlog `--group`'s, based on a suggestion from Jacob
> Stopak.
> 
> The changes are cosmetic, based on the latest round of review from Peff.
> There aren't any substantive changes from the last round.

I looked over the range-diff and gave another careful read of patch 4.
It mostly looks good, though I did find an oddity in patch 4. It's a
really subtle breakage that goes away by the end, so I'm not entirely
sure it's worth dealing with.

So I'd be happy enough with this version, or if you want to do a final
re-roll that addresses that, I'm happy to review it.

-Peff



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux