On Fri, Aug 21, 2020 at 9:19 AM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Jeff King <peff@xxxxxxxx> writes: > > > Yes, I'd agree we should be applying the negative refspecs first, and > > then de-duping / looking for collisions. Which I think is what the patch > > is doing currently. > > Good to see that we thought this through. The reasoning deserves to > be recorded somewhere (perhaps a comment just before making the call > to apply the negative refspec). > > Thanks. I am hoping to add a test case for this as well!