[Reviewers] Changing the name of "reviewers" group and repository

Pierre-Yves David pierre-yves.david at ens-lyon.org
Thu Mar 10 22:21:42 UTC 2016


Good news everyone!

Our reviewers groups start to be in good shape. We have:

- a mailing list,
- a repository waiting to be declared in us (don't use it yet),
   https://www.mercurial-scm.org/repo/reviewers/
- a list of who are in this group on the wiki,
 
https://www.mercurial-scm.org/wiki/ReviewProcess#Push_access_to_the_Clowncopter

The only significant blocker I can currently see is… the name: 
"reviewers". For a long time, the Mercurial project have been claiming 
that "Everyone can do review" and tried to encourage any contributor to 
jump in to help the review flow (and it is somewhat working from time to 
time). Calling the group of people with "push access" the "reviewers" 
send a bad message here. I think we should pick another name, Matt 
seemed to be sympathetic to the idea on IRC.

- ReviewProcess page use "accepters"
- discussion on IRC offered "pushers"
- we also mention "reviewed", but it does not apply to people
- your proposal here

Thoughs? I personally think that "pushers" is a good pick. (with the 
only reservation that it might be confusing with the group of people 
with access to the publishing repository)

Once we pick something, I suggest we:
- rename "repo/reviewers/" to "repo/hg-NAME",
- rename this mailing list to NAME
- deprecate crew and the clowncopter and redirect them to "repo/reviewers/"

In addition to that, either at the sprint, or during the first "steering 
committee" I'm hoping to define some written rules of to get new people 
in the group and also how inactive people get removed from it. But 
that's not blocking to move forward on this topic.

Cheers

-- 
Pierre-Yves David


More information about the Reviewers mailing list