<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 10 December 2014 at 23:21, Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">An idea I've been kicking around would be to *require* Reviewed-by on<br>
all patches.  Now, you'd probably think this would *slow* things down,<br>
but consider this:  If as a patch submitter I know I have to get<br>
someone's review, then I'm going to be more motivated to give other<br>
people reviews myself.  I will also find that by making my patches<br>
easier to review, they'll get a Reviewed-by faster, and thus will land<br>
faster.</blockquote><div><br></div><div>Oooh, whilst we're talking about requirements for merging stuff ... how about enforcing decent Doxygen bits as a hard requirement for merging anything big-ish? Be that internal interfaces inside Weston, or protocol changes.</div><div><br></div><div>Cheers,</div><div>Daniel</div></div></div></div>