<div dir="ltr">Sorry, I've been making a habit of this.<div><br></div><div>I thought the consensus was approve after one review except for 'large changes'.</div><div><br></div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Tue, Jul 9, 2013 at 7:16 PM, Daniel van Vugt <span dir="ltr"><<a href="mailto:daniel.van.vugt@canonical.com" target="_blank">daniel.van.vugt@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I guess then, I would only ask mir-team to not be overconfident. We all make mistakes and we all fail to see some mistakes in proposals. It's best to have more pairs of eyes.<br>
<br>
Waiting a bit longer won't affect throughput of landings, only latency.<div class="im"><br>
<br>
<br>
On 10/07/13 10:12, Robert Ancell wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
My thoughts on this are:<br>
<br>
- If you are in ~mir-devel you are trusted to push changes into lp:mir.<br>
You take responsibility for the changes you make.<br>
- We don't want to push directly since this doesn't guarantee that the<br>
test cases have been run for that commit, so we use merge proposals.<br>
- Merge proposals are a tool to increase quality, not roadblocks to<br>
getting changes in.<br>
- The person who sets the merge proposal to "Approved" is taking<br>
responsibility for that change. It can be a reviewer, or the proposer.<br>
- There's no fixed number of reviews required. It's the responsibility<br>
of the final approver to decide if there has been sufficient review of<br>
the change. If you're in ~mir-devel, you are considered capable of<br>
judging how much review is needed.<br>
<br>
--Robert<br>
<br>
<br>
On Tue, Jul 9, 2013 at 5:19 PM, Daniel van Vugt<br></div>
<<a href="mailto:daniel.van.vugt@canonical.com" target="_blank">daniel.van.vugt@canonical.com</a> <mailto:<a href="mailto:daniel.van.vugt@canonical.com" target="_blank">daniel.van.vugt@<u></u>canonical.com</a>>><div class="im">
<br>
wrote:<br>
<br>
    All,<br>
<br>
    I noticed some people are regularly only waiting for a single<br>
    approval on their MPs and then top-approve themselves.<br>
<br>
    I think top-approving your own MPs is OK if there's already two or<br>
    more reviews from others, and it's been idle for a few days. But<br>
    merging with only one real review, on proposals which are not time<br>
    critical, is probably not ideal. I think we should always be waiting<br>
    for at least a second review, unless it's a time critical issue.<br>
<br>
    Maybe others in mir-team disagree?<br>
<br>
    - Daniel<br>
<br>
    --<br>
    Mir-devel mailing list<br></div>
    <a href="mailto:Mir-devel@lists.ubuntu.com" target="_blank">Mir-devel@lists.ubuntu.com</a> <mailto:<a href="mailto:Mir-devel@lists.ubuntu.com" target="_blank">Mir-devel@lists.<u></u>ubuntu.com</a>><div class="im">
<br>
    Modify settings or unsubscribe at:<br></div>
    <a href="https://lists.ubuntu.com/__mailman/listinfo/mir-devel" target="_blank">https://lists.ubuntu.com/__<u></u>mailman/listinfo/mir-devel</a><br>
    <<a href="https://lists.ubuntu.com/mailman/listinfo/mir-devel" target="_blank">https://lists.ubuntu.com/<u></u>mailman/listinfo/mir-devel</a>><br>
<br>
<br>
</blockquote><div class="HOEnZb"><div class="h5">
<br>
-- <br>
Mir-devel mailing list<br>
<a href="mailto:Mir-devel@lists.ubuntu.com" target="_blank">Mir-devel@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/mir-devel" target="_blank">https://lists.ubuntu.com/<u></u>mailman/listinfo/mir-devel</a><br>
</div></div></blockquote></div><br></div>