Closed Bug 495867 Opened 15 years ago Closed 15 years ago

Update Policy on Getting Commit Access re employers

Categories

(mozilla.org :: Governance, task)

All
macOS
task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mitchell, Assigned: mitchell)

References

()

Details

Attachments

(2 files, 1 obsolete file)

Remove the "Not your employer" requirement for the people sponsoring CVS access.  Address the concern over "group-think" by making sure the super-reviewer will have a fresh perspective.  See discussions in .governance starting 2/4/09 by robert o'callahan and by me on 5/21/09.
Attached is an updated version of the Becoming A Mozilla Committer page.  I tried to also do a diff, but the changes were different enough that it showed the whole page as being new.  I can take another pass at that and see if I can get something that shows the differences.
In case a diff remains hard, here are  the main areas that have changed: 

1.  the introductory paragraphs before the "STOP" notice
2.  the "General Rule"  -- changes SR criteria by removing "different module" language and replacing with "must not have previously reviewed your patches" and ads rationale; removes the "not your employer" requirement
3.  "Modules Not Associated with . . . "  This has changed to be clear that Calendar will now need SR OK for commit access, and then Calendar-focused contributors are full-fledged contributors to comm-central.
4.  Revoking Accounts section added
5.  Dormant Accounts section added
I cleaned up the formatting of the page and generated a diff of the changes.
s/moz-central/mozilla-central/g
Comment on attachment 381600 [details] [diff] [review]
Patch for Becoming A Mozilla Committer page showing differences with current version

>+    project-wide requirements. Fore example, we might decide that each repository should

s/Fore/For/
Revised language for initial section on dormant accounts.  We'll start with this and make the rest of the revised policy official while we figure out the best mechanism -- see discussion on .governance of the last ew days


Dormant Accounts:

As of June 2009 we're working on a plan for addressing accounts that have seen no activity for some specified period of time.  We may disable such accounts.  Assuming we do this,  the process for reactivating them will be a much more lightweight process than that for obtaining commit access the first time.
Attached is a new patch that addresses the last three comments.
Attachment #381600 - Attachment is obsolete: true
Attachment #382303 - Attachment is patch: true
Attachment #382303 - Attachment mime type: application/octet-stream → text/plain
Because people are like that, the question of whether "The super-reviewer must not have previously reviewed your patches" actually means "super-reviewed" or "reviewed or super-reviewed" is going to come up.
Phil

yes you're right, probably best to be clear about this.
Re comment #8, is there a suggestion for new text to use for that sentence?
phil's language seems right:

change "previously reviewed your patches" to "previously reviewed OR SUPER-REVIEWED" your patches

[caps are for emphasis, not for the final doc]
OK, I made this change and checked in the update.  It should be live on the site in about 15 or 20 minutes.  Closing as fixed.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
agh, found another typo, where "moz-central" should be "mozilla-central"

Logistics

For those in a hurry, here's a list of the steps that need to happen to become a Mozilla committer to the moz-central
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
OK, this has been fixed and checked in.
Status: REOPENED → RESOLVED
Closed: 15 years ago15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: