<p dir="ltr">👍👍👍</p>
<br><div class="gmail_quote"><div dir="ltr">On Fri 28 Oct, 2016, 10:03 PM Matthew Turk, <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br class="gmail_msg">
<br class="gmail_msg">
On Fri, Oct 28, 2016 at 12:32 PM, Nathan Goldbaum <<a href="mailto:nathan12343@gmail.com" class="gmail_msg" target="_blank">nathan12343@gmail.com</a>> wrote:<br class="gmail_msg">
> Hi all,<br class="gmail_msg">
><br class="gmail_msg">
> I've noticed recently a distinct downturn in the number of pull request<br class="gmail_msg">
> reviews. That's not to implicate anyone and is not a value judgement on<br class="gmail_msg">
> anyone's contribution. It simply reflects that people have other things to<br class="gmail_msg">
> work and focus on. I happily acknowledge that doing code review regularly is<br class="gmail_msg">
> a tall order.<br class="gmail_msg">
><br class="gmail_msg">
> I'd like to make it so there's more of a culture of doing "drive-by" code<br class="gmail_msg">
> review. I think one of the main reasons only a few people regularly do code<br class="gmail_msg">
> review is that only a few people want to brave the e-mail onslaught and<br class="gmail_msg">
> subscribe to all pull request notifications on the yt_analysis/yt<br class="gmail_msg">
> repository.<br class="gmail_msg">
><br class="gmail_msg">
> Instead, I'd like to propose a hopefully lower noise alternative: forward<br class="gmail_msg">
> notifications about new pull requests to this mailing list. I think it's<br class="gmail_msg">
> straightforward to add this via a webhook on the yt_analysis/yt repository.<br class="gmail_msg">
><br class="gmail_msg">
> We already have similar notifications when an issue is created. This is good<br class="gmail_msg">
> and it encourages more people to look at new issues and offer a helping hand<br class="gmail_msg">
> if they know what to do. I think having a similar setup for pull requests<br class="gmail_msg">
> will encourage more people to regularly pop in and take note of new code<br class="gmail_msg">
> contributions since they'll be newly aware that they are happening.<br class="gmail_msg">
><br class="gmail_msg">
> Let me know what you think, especially if you are against the proposal.<br class="gmail_msg">
><br class="gmail_msg">
> -Nathan<br class="gmail_msg">
><br class="gmail_msg">
> _______________________________________________<br class="gmail_msg">
> yt-dev mailing list<br class="gmail_msg">
> <a href="mailto:yt-dev@lists.spacepope.org" class="gmail_msg" target="_blank">yt-dev@lists.spacepope.org</a><br class="gmail_msg">
> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br class="gmail_msg">
><br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
yt-dev mailing list<br class="gmail_msg">
<a href="mailto:yt-dev@lists.spacepope.org" class="gmail_msg" target="_blank">yt-dev@lists.spacepope.org</a><br class="gmail_msg">
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br class="gmail_msg">
</blockquote></div>