Merging code to PQM - update

Alexander Belchenko bialix at ukr.net
Mon May 3 15:19:01 BST 2010


Jelmer Vernooij пишет:
> On Mon, 2010-05-03 at 08:14 -0500, John Arbash Meinel wrote:
>>> Otherwise, when things were working properly, the queue feature was
>>> really nice to use.
>> I found the # of emails generated to be particularly annoying, but I
>> seem to be the only one who made much of a noise about it. With the
>> email interface and pqm-submit, you can have as little as 3 emails,
>> submission, review + 'merge:approve', noreply@ saying the branch has
>> been merged. With the queue approach, it added quite a few more, with
>> 'setting commit message', 'submitted to pqm', 'approved => queued',
>> 'queued => (not set)', etc.
>>
>> Some could probably be collapsed (setting commit message + submitted +
>> queued, though queued is a diff process.) Similar to how it would be
>> nice if there was the equivalent of (review: approve, merge: approve),
>> so that it could only send 1 email when reviewing via the web.
>>
>> Though I guess I complained and then the accidentally broke my
>> review-via-email :). So maybe I should just keep quite.
> FWIW I also found the amount of email a bit annoying. I'm working around
> that by simply marking whole threads from merge requests I'm not
> involved in as read, but that's not ideal.

The same here.

> 
> Cheers,
> 
> Jelmer




More information about the bazaar mailing list