<div dir="ltr">And if it's proving increasingly difficult to get three approvals for new features, maybe we should relax that requirement as well?</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 14, 2017 at 11:17 AM, Nathan Goldbaum <span dir="ltr"><<a href="mailto:nathan12343@gmail.com" target="_blank">nathan12343@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi John,<div><br></div><div>A number of those are bugfixes, which I think we should feel comfortable to merge with two approves. I've been doing most of the PR merging lately, please feel free to hit the merge button yourself.</div><div><br></div><div>-Nathan</div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jun 14, 2017 at 11:13 AM, John Zuhone <span dir="ltr"><<a href="mailto:jzuhone@gmail.com" target="_blank">jzuhone@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div style="word-wrap:break-word">Hi folks,<div><br></div><div>We have a number of PRs that are nearly ready to go but need some more review. Most of them (but not all) already have two approves and they have docs and tests completed.</div><div><br></div><div>The ones I identify which qualify are:</div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1446" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1446</a></div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1435" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1435</a></div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1433" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1433</a></div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1430" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1430</a></div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1422" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1422</a></div><div><br></div><div><a href="https://github.com/yt-project/yt/pull/1421" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1421</a></div><div><br></div><div>If we could get some help reviewing these and getting them merged in that would be great.</div><div><br></div><div>Best,</div><div><br></div><div>John</div><div><br></div></div><br></div></div>______________________________<wbr>_________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/lis<wbr>tinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>