<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>153591</bug_id>
          
          <creation_ts>2016-01-28 05:49:55 -0800</creation_ts>
          <short_desc>https://webkit.org/team should list only active committers and reviewers.</short_desc>
          <delta_ts>2018-01-31 23:26:10 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Website</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=115387</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=163318</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Konstantin Tokarev">annulen</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>dbates</cc>
    
    <cc>jond</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1159698</commentid>
    <comment_count>0</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-01-28 05:49:55 -0800</bug_when>
    <thetext>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 &quot;active&quot;[1]), and create separate page, e.g. &quot;Hall of Fame&quot;, listing inactive contributors for historical reasons.

[1] https://webkit.org/commit-and-review-policy/#inactive-committer-or-reviewer-status</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1159704</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2016-01-28 07:13:02 -0800</bug_when>
    <thetext>That&apos;s a good idea. Would you be willing to work on a script that analyzes activity?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1159706</commentid>
    <comment_count>2</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-01-28 07:17:25 -0800</bug_when>
    <thetext>Couldn&apos;t we just get away with new field in contributors.json, updated manually? If there is formal procedure of regaining active status, it shouldn&apos;t be managed by bots IMO.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1159827</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2016-01-28 11:54:14 -0800</bug_when>
    <thetext>I agree that it&apos;s debatable whether this should be done automatically. However we&apos;d still need to automate gathering the data - it&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1187510</commentid>
    <comment_count>4</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-26 12:56:22 -0700</bug_when>
    <thetext>Looks like the right place of this functionality is validate-committer-lists, it already does some analysis of commit activity</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1395355</commentid>
    <comment_count>5</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2018-01-31 23:21:28 -0800</bug_when>
    <thetext>(In reply to Alexey Proskuryakov from comment #3)
&gt; I agree that it&apos;s debatable whether this should be done automatically.
&gt; However we&apos;d still need to automate gathering the data - it&apos;s not feasible
&gt; to manually go over the whole list of contributors checking when their
&gt; latest commit (or bugzilla comment) was.
&gt; 

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

    </bug>

</bugzilla>