<div dir="ltr"><div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On 9 Oct 2017 10:59, "Marco Martin" <<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail-m_-4585948663864042724m_1254475808074819561gmail-m_1700561216612542226quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-m_-4585948663864042724m_1254475808074819561gmail-m_1700561216612542226elided-text">On Sun, Oct 8, 2017 at 12:01 PM, David Edmundson<br>
<<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>> wrote:<br>
> Martin,<br>
><br>
> don't go out of your way to get offended, most of the criticisms (of which<br>
> certainly 1 part is valid) are directed at a few certain people in Plasma.<br>
> Myself being one; Ben's kindly going out of his way to not publicly name<br>
> names.<br>
<br>
</div>yet.. being that vague now i'm not sure what the exact issues are (and<br>
if i caused some of them, which i fear.. but i don't know)<br>
all i care at the moment is just to get the issues fixed<br><br>
--<br>
Marco Martin<br></blockquote></div></div></div><div dir="auto"><br></div>On communication problems.</div><div dir="auto"><br></div><div>So we had Marco not see your comment on a Kirigami review, but I got a similar situation today. It's pretty easy to see why we miss stuff.<br></div><br><div dir="auto"><div>The email I got yesterday literally has just the following 2 lines.<br></div><div><br></div><div>>bcooksley reopened this revision.<span class="gmail-m_-4585948663864042724m_1254475808074819561gmail-im"><br>>This revision is now accepted and ready to land.</span></div><span class="gmail-m_-4585948663864042724m_1254475808074819561gmail-im"></span></div><div dir="auto"><span class="gmail-m_-4585948663864042724m_1254475808074819561gmail-im"><br></span></div><div>It's at best confusing. It doesn't exactly look like a problem.<br></div><div><br></div><div>(Then that again for another email, and then the actual comment 3 emails later...)<br></div><br><div>Turning this into useful feedback, when we have an issue 
with a commit that's committed, I think it'd better to not bump the review, but comment on the 
commit in diphusion so it starts a new thread. It should be one less lookup from the sysadmin side too.</div><br><div>David<br></div>
</div>