On Sun, Jan 20, 2013 at 1:24 PM, David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

This is in response to a comment I saw on reviewboard last night, but<br>
I want to reply to the whole mailing list.<br>
<br>
If you've ever committed code on KTp, you should be helping do reviews<br>
and you should click "ship it!" if you understand the code and agree<br>
with the patch. I don't want people to think "I'm not the manager, I<br>
shouldn't be clicking Ship it". In KTp, all developers are equal, and<br>
should all have equal powers & responsibility.<br>
<br>
If I write a patch, and I want a specific person to review it, I'll<br>
simply wait for a second "ship it" from someone else. If you review<br>
someone elses code and click ship it! and I disagree, I'll simply<br>
comment afterwards. Even if the code gets commmitted, I'll still see<br>
the review and can fix it afterwards.<br>
<br>
David.<br>
_______________________________________________<br>
KDE-Telepathy mailing list<br>
<a href="mailto:KDE-Telepathy@kde.org">KDE-Telepathy@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-telepathy" target="_blank">https://mail.kde.org/mailman/listinfo/kde-telepathy</a><br>
</blockquote></div><br><div>I get the message. :P</div><div><br></div><div>Thanks!</div><div>Aleix</div>