[scikit-learn] Assigning reviewers to PRs

[Adrin]

Hi Madhura,

We don't really follow the assignment workflow. Whichever of the
maintainers is free and feels like reviewing the PR
will do so. But in general review time is our main bottleneck, and it may
take some time for us to get on a PR.

Patience and every now and then writing a comment under your PR saying "a
gentle ping" or something
would probably remind us that it's still not reviewed :)

Thanks for contributing,
Adrin.

On Thu, Jan 23, 2020 at 6:33 AM Madhura Jayaratne <madhura.cj at gmail.com>
wrote:

> Hi Scikit-learn team,
>
> I have submitted a PR implementing support for ICE plots at [1]. There I
> noticed that Github suggests reviewers based on the recent edits and
> reviews to the files changed in the PR. I am wondering what is the
> preferred practice for assigning reviewers. (Initially, I assigned Guillaume
> Lemaitre without thinking much, but he cannot review at the moment) I am
> wondering, should I assign someone from the suggested list or should I
> wait for the core developers to assign themselves?
>
> [1] https://github.com/scikit-learn/scikit-learn/pull/16164
>
> --
> Thanks and Regards,
>
> Madhura Jayaratne
>
> _______________________________________________
> scikit-learn mailing list
> scikit-learn at python.org
> https://mail.python.org/mailman/listinfo/scikit-learn
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/scikit-learn/attachments/20200123/3d158cf1/attachment.html>