Lgtm Am 09.01.2018 8:24 vorm. schrieb "YiZhi Liu" <javeli...@gmail.com>:
> +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 >