On Fri, Apr 20, 2012 at 04:59:00PM +0200, Matthieu Moy wrote: > OK, so this v2 is not supposed to be a draft anymore. It has > documentation (while I was there, I added PATCH 1/4 thas tries to > document better the existing modes), and I removed the hunks that came > here after a broken merge resolution. > > This is based on next, it at least requires 135dade that creates > t5528. You are better off to just build it on 135dade in that case, as that is what Junio will apply it on (never directly on top of next). It generally isn't a problem, but there's no point reason not to do so. > Clemens Buchacher (1): > t5570: use explicit push refspec > > Matthieu Moy (3): > Documentation: explain push.default option a bit more > push: introduce new push.default mode "simple" > push: start warning upcoming default change for push.default I commented separately on patches 1 and 2, but modulo those minor comments, the series looks OK to me. The "Git 2.0" mention in patch 4/4 might need to be tweaked. :) -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html