openstack/governance
Code Issues Proposed changes

Exclude inactive core reviewers from core metrics

Browse Source 
The single-vendor and diverse-affiliation tags include inactive core
reviewers in the metrics for core stats. Some projects are more
proactive in their core-reviewers list maintenance, whereas others tend
to stall for a while even though some of the core reviewers have not
been as active/focused as they once were.

Therefore, it's important to differentiate between active and inactive
core reviewers. Non active core reviewers with just a handfull of
reviews would send a wrong message as far as diversity and affiliation
activity goes.

This commit takes the first step towards a more accurate metric by
stating that only active members should be counted, while the actual
definition and exact threshold are discussed in follow-up patches.

Change-Id: I7db6db7626616c39ccb17bd7229a740065de5e67
This commit is contained in:
Flavio Percoco 2016-06-21 16:31:29 +02:00
parent 2cef824179
commit 1ab5e80ba8
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
reference/tags/team_diverse-affiliation.rst
View File

@ -53,11 +53,11 @@ should represent >80% of any of the following:
* the sum of all reviews done against patches submitted to any of the git
repositories managed by the team
* the sum of all reviews done by core reviewers against patches submitted to any
of the git repositories managed by the team
* the sum of all reviews done by active core reviewers against patches submitted
to any of the git repositories managed by the team
* the union of the memberships of the core review teams associated with the git
repositories managed by the team
* the union of the active memberships of the core review teams associated with
the git repositories managed by the team
The timeline used for evaluation is based on the past 6 months, and should be
updated around the same time as the 6 month release.

8
reference/tags/team_single-vendor.rst
View File

@ -54,11 +54,11 @@ any of the following over the prior six months:
* the sum of all reviews done against patches submitted to any of the git
repositories managed by the team
* the sum of all reviews done by core reviewers against patches submitted to any
of the git repositories managed by the team
* the sum of all reviews done by active core reviewers against patches submitted
to any of the git repositories managed by the team
* the union of the memberships of the core review teams associated with the git
repositories managed by the team
* the union of the active members of the core review teams associated with the
git repositories managed by the team
The application of this tag to new projects should be updated around the same
time as the 6 month release. But the removal of this tag should happen shortly