Re: [PATCH/RFC] improve no-op push output

From: Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date: May 31, 2012, 06:10 UTC
Message-ID: <vpqtxywswwo.fsf@bauges.imag.fr>
In-Reply-To: <7vr4u1zhcz.fsf@alter.siamese.dyndns.org>

Junio C Hamano <gitster@pobox.com> writes:

Show 11 quoted lines

>> would just print:
>>
>>   Everything up-to-date
>>
>> and now you get:
>>
>>   To /tmp/push-message/parent
>>    = [up to date]      master -> master
> [...]
>
> 	Everything up-to-date (the current branch not pushed)

I like both. They don't really take more screen space (1 more line for Jeff, which is acceptable IMHO), and do add information.

I'd add the number of branches pushed in case it's not one, so it could say one of:

  To /tmp/push-message/parent
   = [up to date]      master -> master

or

  Everything up-to-date (X branches pushed, including the current)

or

  Everything up-to-date (X branches pushed, not including the current)

It would give a clue to a user having created many local branches that his local branches were not pushed, and vice-versa, may help people who did not understand that "push.default=matching" was pushing multiple branches.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/