Comment 8 for bug 492145
Revision history for this message
| Andrew Bennetts (spiv) wrote Re: [Bug 492145] Re: email about updated mp diffs includes obsolete/misleading cover letter | #8 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
Martin Pool wrote:
> So I think this bug is: when you are requested to review a mp, you are
> always sent an email with the contents of the first message, even if
> there has been extensive discussion and updates to the branch such
> that the first message is no longer a good representation of what's
> going on.
>
> Bugs largely avoid this by having an editable summary and description.
That's true. Another option (which could complement editable cover
letters) might be to include an activity summary along the lines of “1
Approve, 2 Needs Fixing, and 6 Comments”. That would give you a pretty
good sense that you're diving into the middle of an active discussion,
rather than at the start. The +activereviews page has text like that
and it is a useful indication.