[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mercurial-devel
Subject:    Re: [PATCH] Tweaks to MQ user interface
From:       Dirkjan Ochtman <dirkjan () ochtman ! nl>
Date:       2009-06-28 19:08:21
Message-ID: ea2499da0906281208u41b57732w413295609a13ea87 () mail ! gmail ! com
[Download RAW message or body]

On Sun, Jun 28, 2009 at 17:22, Dan Villiom Podlaski
Christiansen<danchr@gmail.com> wrote:
> Attached below are three small patches for the MQ and colour extensions:

Hi Dan,

Thanks for working on these. First of all, could you please send your
patches using patchbomb with inline patches, as is customary on this
list? It makes reviewing massively easier. Also, since we're currently
in feature freeze, we won't be able to take your patches right now.
Please resend after Mercurial 1.3 is released.

Cheers,

Dirkjan

> The first patch changes the index column in the verbose output of ‘qseries’
> to be right aligned, and also crops the line interactive mode.
>
> The second patch changes the colorisation of ‘qseries’ when passed
> -v/--verbose or -s/--summary, such that only the patch name is coloured. I
> find this more readable this way. It also adapts the extension to the
> altered output caused by the first patch.
>
> The third patch is also quite simple in functionality, but rather invasive
> in how it affects the user interface. It adds a wrapper to all built-in
> commands which require a repository, and add a -Q/--queue option which
> causes them to operate on a queue repository, if present. The error message
> for an absent queue repository was copied from another location in the
> extension.
>
> If the third patch is accepted, it might be worthwhile to either deprecate
> or hide the ‘qcommit’ command. It'd be one less command cluttering the help
> screen :)
>
> I started a test run with of patches about 20 minutes ago: so far,
> everything seems fine.
>
>> 2009-06-27  Dan Villiom Podlaski Christiansen  <danchr@gmail.com>
>>
>>        * hgext/mq.py:
>>        mq: align columns in verbose qseries output.
>>        [bbe5d7648c5b] [qbase, mq-qseries-align.diff]
>>
>>        * hgext/color.py:
>>        color: don't highlight the summary part of the qseries command
>>        [c95c14547c63] [mq-qseries-color.diff]
>>
>> 2009-06-28  Dan Villiom Podlaski Christiansen  <danchr@gmail.com>
>>
>>        * hgext/mq.py:
>>        mq: add -Q/--queue option to some commands for operating on the
>>        queue repo.
>>        [99c4b776e36a] [qtip, tip, mq-repo-option.diff]
>
> --
>
> Dan Villiom Podlaski Christiansen
> danchr@gmail.com
>
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel@selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel
>
>

_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@selenic.com
http://selenic.com/mailman/listinfo/mercurial-devel

[prev in list] [next in list] [prev in thread] [next in thread]