[Neuroimaging] Upcoming nipy release

[JB Poline]

Sounds very reasonable to me!
JB

On Fri, Sep 25, 2015 at 2:30 AM, Yaroslav Halchenko
<lists at onerussian.com> wrote:
>
> On Thu, 24 Sep 2015, Matthew Brett wrote:
>> >> Our setup is https://github.com/datalad/buildbot which is in large based on
>> >> https://github.com/ethereum/ethereum-buildbot and apparently work on supporting
>> >> pull_requests in stock buildbot since then was accepted!
>> >> https://github.com/buildbot/buildbot/pull/1632
>> >> so I might look into redoing it using stock features
>
>> > That would be really good - if you do have time to do it.  It would be
>> > ideal to get the buildbots integrated into the github interface...
>
>> After thinking about it for a little bit, I was worried about a)
>> overwhelming the buildbot machines, and b) allowing any PR to execute
>> code on the buildbot machines.
>
> our setup restricts automagic testing of PRs only for a limited set of
> github logins.  For the rest of PRs I (or other developers with enough
> rights) would need to add a label 'buildbot' to the PR to trigger
> buildbot considering it (so similar to homu's approvals)
>
>> I wonder whether we could use Homu
>> instead?
>> https://github.com/barosl/homu
>
> I thought I saw all of the possible solutions when I was setting it up a
> while back, may be Homu too... there seemed to be quite a bit of new
> development in it since then.
>
>> I think this would mean that approved reviewers could mark the PR with
>> a comment like '@homu-user r+' to the PR, and this could trigger
>> builds both on travis-ci, and the buildbots, and the PR would only get
>> merged if the all the tests pass.  I think.
>
>> That seems like a good mix - an approved reviewer has to OK it before
>> it is tested on the buildbots and travis before merging.
>
>> What do you think?
>
> yeah -- sounds good, but on the other hand requires additional
> interaction (unless there are automatic "approvals" for testing for some
> logins). it is for sure probably better than the ad-hoc solution I
> adopted, but it might still be better to re-review what is the built-in
> support within buildbot atm.  It might be easier to provide similar
> logic if not present "natively"
>
> --
> Yaroslav O. Halchenko
> Center for Open Neuroscience     http://centerforopenneuroscience.org
> Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
> Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
> WWW:   http://www.linkedin.com/in/yarik
> _______________________________________________
> Neuroimaging mailing list
> Neuroimaging at python.org
> https://mail.python.org/mailman/listinfo/neuroimaging