From: Junio C Hamano Date: Wed, 20 Mar 2013 18:53:44 GMT Subject: Re: [PATCH 5/6] remote.c: introduce remote.pushdefault Message-ID: <7vk3p1ewcn.fsf@alter.siamese.dyndns.org> In-Reply-To: <20130320183255.GL3655@google.com> Jonathan Nieder writes: > Ramkumar Ramachandra wrote: > >> This new configuration variable defines the default remote to push to, >> and overrides `branch..remote` for all branches. > > Micronit: I think this would be easier to explain if it came after > patch 6, since then you could say "In other words, it is a default for > branch..pushremote for all branches" and readers would not have > to wonder "Why does the more generic configuration override the more > specific one?". > > My two cents, > Jonathan Thanks for all good comments (not only to this one but to others in the series). I agree with all of them.