[yt-dev] Forward pull request created e-mails to the yt-dev mailing list

[Wise, John H]

+1

--
John Wise
Associate Professor of Physics
Center for Relativistic Astrophysics, Georgia Tech
http://cosmo.gatech.edu<http://cosmo.gatech.edu/>

________________________________
From: Nathan Goldbaum <nathan12343 at gmail.com>
Sent: Oct 28, 2016 12:34
To: yt-dev at 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.

-Nathan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20161028/e986c59d/attachment-0001.htm>