Re: [PATCH 10/10] lower core.maxTreeDepth default to 2048
- From
Jeff King <peff@peff.net>
- Date
- Aug 31, 2023, 17:42 UTC
- Message-ID
- <20230831174215.GA3208283@coredump.intra.peff.net>
- In-Reply-To
- <ZPBt6VYAWHJhv3N2@ugly>
On Thu, Aug 31, 2023 at 12:39:37PM +0200, Oswald Buddenhagen wrote:
Show 7 quoted lines
> On Thu, Aug 31, 2023 at 02:23:20AM -0400, Jeff King wrote: > > But I thought that > > following the sequence of logic (from "4096 is probably OK" to "whoops, > > it's not") had some value to share. > > > of course, but you can just integrate that into the squashed commit message. > having it all in one place makes it easier to follow.
Yes, though I think having it as a separate patch makes it easier to revisit later (e.g., by reverting or by replacing the patch during a re-roll).
-Peff