+1 on new PRs to this list.

On Fri, Oct 28, 2016 at 10:08 AM, Wise, John H <jwise@physics.gatech.edu> wrote:


John Wise 
Associate Professor of Physics 
Center for Relativistic Astrophysics, Georgia Tech 

From: Nathan Goldbaum <nathan12343@gmail.com>
Sent: Oct 28, 2016 12:34
To: yt-dev@lists.spacepope.org
Subject: [yt-dev] Forward pull request created e-mails to the yt-dev mailing list

Hi all,

I've noticed recently a distinct downturn in the number of pull request reviews. That's not to implicate anyone and is not a value judgement on anyone's contribution. It simply reflects that people have other things to work and focus on. I happily acknowledge that doing code review regularly is a tall order.

I'd like to make it so there's more of a culture of doing "drive-by" code review. I think one of the main reasons only a few people regularly do code review is that only a few people want to brave the e-mail onslaught and subscribe to all pull request notifications on the yt_analysis/yt repository.

Instead, I'd like to propose a hopefully lower noise alternative: forward notifications about new pull requests to this mailing list. I think it's straightforward to add this via a webhook on the yt_analysis/yt repository.

We already have similar notifications when an issue is created. This is good and it encourages more people to look at new issues and offer a helping hand if they know what to do. I think having a similar setup for pull requests will encourage more people to regularly pop in and take note of new code contributions since they'll be newly aware that they are happening.

Let me know what you think, especially if you are against the proposal.


yt-dev mailing list

Cameron Hummels
NSF Postdoctoral Fellow
Department of Astronomy
California Institute of Technology