Re: [PATCH 0/8] tightening ref handling outside of refs/
- From
Jeff King <peff@peff.net>
- Date
- Apr 30, 2024, 10:45 UTC
- Message-ID
- <20240430104530.GG1279403@coredump.intra.peff.net>
- In-Reply-To
- <Zi9oRuknlhfG1-WX@tanuki>
On Mon, Apr 29, 2024 at 11:28:38AM +0200, Patrick Steinhardt wrote:
Show 15 quoted lines
> On Mon, Apr 29, 2024 at 04:42:38AM -0400, Jeff King wrote: > > On Mon, Apr 29, 2024 at 04:15:13AM -0400, Jeff King wrote: > > > > > [1/8]: t0600: don't create ref outside of refs/ > > > [2/8]: t5619: use fully qualified refname for branch > > > > You can probably guess that I found these test cleanups only after > > writing the rest of the series and seeing them fail. :) > > > > It turns out there is one more spot that is run only with reftables (so > > CI caught it, but my local testing did not): > > Yeah, that's an issue by itself in my opinion. It's ultimately the > reason why I change this to always run the backend-specific tests in > [1].
Ah, I hadn't seen that series. Yes, I'd be very much in favor of that change.
-Peff