Bug 134403 – Update policy for Inactive Committer and Reviewer status

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Website

[Bem Jones-Bey]

Update policy for Inactive Committer and Reviewer status

[Bem Jones-Bey]

Created attachment 233996 [details]
Patch

[Beth Dakin]

Comment on attachment 233996 [details]
Patch

This looks good to me!

[Ryosuke Niwa]

Comment on attachment 233996 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=233996&action=review

> Websites/webkit.org/coding/commit-review-policy.html:108
> +landing at least one patch in the past year.</p>

or reviewing a patch for the reviewer status.

> Websites/webkit.org/coding/commit-review-policy.html:111
> +<p>Inactive Committers can regain Active Committer status by simply landing a
> +non-trivial patch and asking to return to Active status.</p>

We might want to separately mention that subversion account not used to commit patches into the repository for more than one year will be deactivated for security reasons.

[Holger Freyther]

What is the issue you are trying to solve? I am a WebKit reviewer, I look at the commits from time to time, I read the webkit-dev mailinglist but I have not reviewed or landed a patch in a very long time. Given the above definition I count as inactive contributor.

On the other hand I am certainly not qualified to land or review patches right now. For me it would be natural to get more familiar with the code before landing patches again.

[Bem Jones-Bey]

Created attachment 234015 [details]
Patch

[Bem Jones-Bey]

(In reply to comment #3)
> (From update of attachment 233996 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=233996&action=review
> 
> > Websites/webkit.org/coding/commit-review-policy.html:108
> > +landing at least one patch in the past year.</p>
> 
> or reviewing a patch for the reviewer status.
> 
> > Websites/webkit.org/coding/commit-review-policy.html:111
> > +<p>Inactive Committers can regain Active Committer status by simply landing a
> > +non-trivial patch and asking to return to Active status.</p>
> 
> We might want to separately mention that subversion account not used to commit patches into the repository for more than one year will be deactivated for security reasons.

I've uploaded a new version that incorporates these comments.

[Bem Jones-Bey]

(In reply to comment #4)
> What is the issue you are trying to solve? I am a WebKit reviewer, I look at the commits from time to time, I read the webkit-dev mailinglist but I have not reviewed or landed a patch in a very long time. Given the above definition I count as inactive contributor.
> 
> On the other hand I am certainly not qualified to land or review patches right now. For me it would be natural to get more familiar with the code before landing patches again.

The only issue I personally am trying to solve is that for new people it can be hard to know who to talk to for reviews, since the reviewer list is long and lists many people that are no longer even as active as you are. It would satisfy me if we just made it so that these inactive reviewers were listed separately on the team.html page.

There was a security concern about having lots of inactive subversion accounts, and even with having inactive accounts that have access to things like EWS. I believe that's the main point in this, but hopefully some of those that feel more strongly about it than I can elaborate.

[Bem Jones-Bey]

Created attachment 234565 [details]
Patch

[Nico Weber]

Comment on attachment 234565 [details]
Patch

I feel it's in the spirit of this discussion if this is r+d by a (now no longer :-P) inactive reviewer.

[WebKit Commit Bot]

Comment on attachment 234565 [details]
Patch

Clearing flags on attachment: 234565

Committed r170904: <http://trac.webkit.org/changeset/170904>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryosuke Niwa]

Comment on attachment 234565 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=234565&action=review

> Websites/webkit.org/coding/commit-review-policy.html:127
> +Reviewer can send an email to webkit-reviewers requesting it.</p>

This should be webkit-dev as committers can't post messages to webkit-reviewers.

[Bem Jones-Bey]

Comment on attachment 234565 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=234565&action=review

>> Websites/webkit.org/coding/commit-review-policy.html:127
>> +Reviewer can send an email to webkit-reviewers requesting it.</p>
> 
> This should be webkit-dev as committers can't post messages to webkit-reviewers.

Or maybe webkit-committers? (same with committers wanting to be reactivated?)

[Bem Jones-Bey]

(In reply to comment #13)
> (From update of attachment 234565 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=234565&action=review
> 
> >> Websites/webkit.org/coding/commit-review-policy.html:127
> >> +Reviewer can send an email to webkit-reviewers requesting it.</p>
> > 
> > This should be webkit-dev as committers can't post messages to webkit-reviewers.
> 
> Or maybe webkit-committers? (same with committers wanting to be reactivated?)

Haven't heard back, so I've created a patch with this change:
https://bugs.webkit.org/show_bug.cgi?id=135203