[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