From: Jeff King Date: Thu, 31 Aug 2023 17:42:15 GMT Subject: Re: [PATCH 10/10] lower core.maxTreeDepth default to 2048 Message-ID: <20230831174215.GA3208283@coredump.intra.peff.net> In-Reply-To: On Thu, Aug 31, 2023 at 12:39:37PM +0200, Oswald Buddenhagen wrote: > 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