<div dir="ltr"><div>Matthias,</div><div><br></div>Regarding the emails, could this be due to notification settings? I notice that PR reviews are a separate item you have to check in the notification center.<div><br></div><div>Ryan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 19, 2016 at 8:32 PM, Matthias Bussonnier <span dir="ltr"><<a href="mailto:bussonniermatthias@gmail.com" target="_blank">bussonniermatthias@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
One data point, it seem that some users/dev (at least on Jupyter side)<br>
are complaining that they don't get the "grouped-review-comments" by<br>
emails. So are waiting for feedback they never get until they check<br>
the PR.<br>
<br>
Still haven't figured out when this happens.<br>
<br>
I'm curious if the "Approved" review get reset after a new commit is<br>
pushed, and If so and the N reviewer is enforced, it can be annoying<br>
if someone approved with a small condition, like typo fixed or<br>
something similar.<br>
--<br>
M<br>
<div><div class="h5"><br>
On Mon, Sep 19, 2016 at 7:24 PM, Thomas Caswell <<a href="mailto:tcaswell@gmail.com">tcaswell@gmail.com</a>> wrote:<br>
> Folks,<br>
><br>
> Have people had a chance to look at the new review tools on github?  Do we<br>
> want to work these into our workflow?  It looks like we can set it so that<br>
> PRs can only be merged via the web UI with at least one approve review and<br>
> no 'needs work' reviews if we want.   My initial reaction is we should try<br>
> to use the review tools, but hold on on the engineering control on merges<br>
> until we have a better sense of what that would look like.<br>
><br>
> Has anyone looked at the 'projects' feature yet?<br>
><br>
> A suggestion from Nelle Varoquaux is to follow skimage/sklearn and on review<br>
> add a [mrg+N] to the title to help other devs find PRs that are almost ready<br>
> to merge, but need another set of eyes.<br>
><br>
> Tom<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> Matplotlib-devel mailing list<br>
> <a href="mailto:Matplotlib-devel@python.org">Matplotlib-devel@python.org</a><br>
> <a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/matplotlib-<wbr>devel</a><br>
><br>
______________________________<wbr>_________________<br>
Matplotlib-devel mailing list<br>
<a href="mailto:Matplotlib-devel@python.org">Matplotlib-devel@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/matplotlib-<wbr>devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Ryan May<br><br></div></div></div>
</div>