On Mon, Apr 29, 2024 at 11:28:38AM +0200, Patrick Steinhardt wrote: > 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
