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

Cameron Hummels chummels at gmail.com
Fri Oct 28 16:08:38 PDT 2016


+1 on new PRs to this list.

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

> +1
> --
> John Wise
> Associate Professor of Physics
> Center for Relativistic Astrophysics, Georgia Tech
> 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
>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>
>


-- 
Cameron Hummels
NSF Postdoctoral Fellow
Department of Astronomy
California Institute of Technology
http://chummels.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20161028/b020b40b/attachment.html>


More information about the yt-dev mailing list