+1 sounds like a great idea. We also need a mechanism to identify “good reviewers”. Maybe we can count the number of :thumsup: in each review. Or any other better way?
On Fri, Oct 19, 2018 at 8:22 PM Tianqi Chen <tqc...@cs.washington.edu> wrote: > Dear MXNet Community: > > There is a great discussion going on in terms of lowering the barrier of > entries and encourage more contribution to the project. One of the general > goals is to encourage a broader pool of contributions. I want to make the > following proposal: > > Besides Committers and PMC, let us also recognize Reviewers in the > community. This is a "pseudo role" as there is no such official role in > Apache. But I want to explore the possibility of recognising active > reviewers for example, by adding a list of names in the contributor list. > In general, I find it is really helpful to have more code reviews. > Recognising good reviewers early enables us to find candidate for > committers, and encourage them to contribute and understand what is the bar > of code quality that is required to merge the code. > > This can provide the community more evidence when recruiting new > committers. After all committers is about write access to the code and > understand the consequence of the responsibility -- which is usually can be > found in high quality reviews. > > Please let me know what you think. > Tianqi >
