+1 for adding @CodingCat (Nan Zhu) as owner of Scala package.

2018-01-08 21:58 GMT-08:00 Mu Li <muli....@gmail.com>:

> It has been a while for discussing having maintainers for individual
> modules. A maintainer for a module will be the main person who reviews and
> approves PRs contributed to that module.
>
> Currently, some maintainers are listed in the file CODEOWNERS
> <https://github.com/apache/incubator-mxnet/blob/master/CODEOWNERS>:
>
> # Owners of Apache MXNet
> # Global owners
> *            @apache/mxnet-committers
> # Owners of language bindings
> R-package/*        @thirdwing
> scala-package/*        @javelinjs
> perl-package/*        @sergeykolychev
> # CMake owners
> CMakeLists.txt        @cjolivier01
> cmake/*            @cjolivier01
>
> However, this list is incomplete. This document proposes how to partition
> mxnet codes into modules and put tentative maintainers for each module.
>
>    - I tried to select the activate contributor who contributed most to the
>    module, not the new contributor who is going to work for it. But I may
> be
>    outdated.
>    - Code review is not easy. So maintainers may need help at the
>    beginning.
>    - Eric (@piiswrong) did most PR reviews, so I didn't put his name on
>    every module.
>
> Any suggestion is welcome.
> FrontendPYTHON: @szha
>
> python/
>
> R: @thirdwing @hetong007
>
> R-package/
>
> SCALA: @CodingCat @javelinjs
>
> scala-package/
>
> PERL@sergeykolychev
>
> perl-package/
>
> C++?
>
> cpp-package/
>
> MATLAB: DEPRECATE IT?
>
> matlab/
>
> AMALGAMATION: DEPRECATE IT?
>
> amalgamation/
>
> Backend@reminisce @eric-haibin-lin
>
> include/
> src/
>
> Build@cjolivier01
>
> docker/
> docker_multiarch/
> make/
> Makefile
> cmake/
> CMakeLists.txt
> setup-utils/
> prepare_mkl.sh
>
> Test@marcoabreu
>
> tests/
> Jenkinsfile
>
> Doc & Website@kevinthesun
>
> docs/
>
> ExamplesNot sure, since we have a lot of contributors here. We probably
> need to remove low quality examples and assign one maintainer for each of
> the rest.
>
> example/
>
> ToolsNot sure as well, since we have a bunch of things there. Probably need
> to the same thing as examples
>
> tools/
> plugin/
>
> AppendixLines of codes added into each folder on the last two months:
> Lines of codes added into example/
> Lines of codes added into src/
>



-- 
Yizhi Liu
DMLC member
Amazon Web Services
Vancouver, Canada

Reply via email to