Merging code to PQM - update

[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.

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20100503/6d4dc05c/attachment.pgp