Bug 153591 - https://webkit.org/team should list only active committers and reviewers.
Summary: https://webkit.org/team should list only active committers and reviewers.
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Website (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-28 05:49 PST by Konstantin Tokarev
Modified: 2018-01-31 23:26 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin Tokarev 2016-01-28 05:49:55 PST
Right now https://webkit.org/team lists all contributors without any separation between active and inactive folks. It would be much more useful if team page contained only active committers and reviewers (according to current definition of "active"[1]), and create separate page, e.g. "Hall of Fame", listing inactive contributors for historical reasons.

[1] https://webkit.org/commit-and-review-policy/#inactive-committer-or-reviewer-status
Comment 1 Alexey Proskuryakov 2016-01-28 07:13:02 PST
That's a good idea. Would you be willing to work on a script that analyzes activity?
Comment 2 Konstantin Tokarev 2016-01-28 07:17:25 PST
Couldn't we just get away with new field in contributors.json, updated manually? If there is formal procedure of regaining active status, it shouldn't be managed by bots IMO.
Comment 3 Alexey Proskuryakov 2016-01-28 11:54:14 PST
I agree that it's debatable whether this should be done automatically. However we'd still need to automate gathering the data - it's not feasible to manually go over the whole list of contributors checking when their latest commit (or bugzilla comment) was.

There are lots of applications for this data, such as disabling commit access for folks who never commit, thus reducing password brute force attack target.

Any policy changes would need to be widely discussed of course.
Comment 4 Konstantin Tokarev 2016-04-26 12:56:22 PDT
Looks like the right place of this functionality is validate-committer-lists, it already does some analysis of commit activity
Comment 5 Daniel Bates 2018-01-31 23:21:28 PST
(In reply to Alexey Proskuryakov from comment #3)
> I agree that it's debatable whether this should be done automatically.
> However we'd still need to automate gathering the data - it's not feasible
> to manually go over the whole list of contributors checking when their
> latest commit (or bugzilla comment) was.
> 

Bug #115387 has an out-of-date patch to teach webkit-patch how to find emeritus reviewers and committers.