<div class="gmail_quote">On 15 October 2011 22:50, Fernando Perez <span dir="ltr">&lt;<a href="mailto:fperez.net@gmail.com">fperez.net@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div id=":dr">- we need more eyes on the open PRs, and there&#39;s a little bit for<br>
everyone: Qt, console, notebook, internals.  Helping test/review an<br>
existing PR gives the core team more confidence on how to move<br>
forward, and is a great way to get acquainted with the codebase, so<br>
don&#39;t be shy!  Anyone can comment on a PR, and we really value new<br>
pairs of eyes on them.  As a rule of thumb, a PR with 20 or 30<br>
comments is likely to have an already complicated discussion going on<br>
so you may want to skip those on a  first pass, unless you happen to<br>
care/know about the topic.  But one with very little feedback could<br>
really benefit from your eyes, there&#39;s a few that still have 0<br>
comments.</div></blockquote></div><br>I jotted down a quick set of things to look for in a pull request:<br><br><a href="http://wiki.ipython.org/The_perfect_pull_request">http://wiki.ipython.org/The_perfect_pull_request</a><br>

<br>It&#39;s pretty terse at the moment, so feel free to flesh it out.<br><br>Thomas<br>