Quick approval

Alexandros Frantzis alexandros.frantzis at canonical.com
Tue Jul 9 07:16:52 UTC 2013


On Tue, Jul 09, 2013 at 01:19:21PM +0800, Daniel van Vugt wrote:
> All,
> 
> I noticed some people are regularly only waiting for a single
> approval on their MPs and then top-approve themselves.
> 
> I think top-approving your own MPs is OK if there's already two or
> more reviews from others, and it's been idle for a few days. But
> merging with only one real review, on proposals which are not time
> critical, is probably not ideal. I think we should always be waiting
> for at least a second review, unless it's a time critical issue.
> 
> Maybe others in mir-team disagree?
> 
> - Daniel
> 

I agree. If an MP is trivial (a typo, a simple fix), I think it's OK for
the first *reviewer* to top-approve it.

Thanks,
Alexandros



More information about the Mir-devel mailing list