501646 - Reviewers document is out of date in many areas

Status: RESOLVED FIXED

(mozilla.org :: Governance, task)

Description

[Gervase Markham [:gerv]]

http://www.mozilla.org/hacking/reviewers.html may have an up-to-date list of super-reviewers, but lots of other things about it are out of date. There are broken links (SeaMonkey Code Reviewer's Guide, SeaMonkey Engineering Bible), dead mailing lists ([e-mail]) and long-gone people ([e-mail]). The whole thing could do with a spring-clean.

Unless anyone objects, I'll prepare a patch.

Gerv

Comment 1

[Mitchell Baker]

Gerv

mconnor has a partial fix for this; at least for the policy part of it.  The text is in .governance and i think dev.planning.  If you could prepare a patch that incorporated his plans plus fix whatever else you think makes sense that would be awesome.  Then we'll have both a new policy for super-review and a doc that maks sense.

Comment 2

[Gervase Markham [:gerv]]

Yeah, sorry - I got my documents in a muddle and forgot that Mike was updating this one. Re-reading the relevant thread (http://groups.google.com/group/mozilla.governance/browse_thread/thread/1c7c826dca44043c#), I think that his patch probably does all the work and, even if it doesn't, it'll be easier to evaluate what's left to do once it gets checked in.

Mike: I'm happy to leave you to it, or to take your patch and drive it if you are busy. Let me know what's best.

Gerv

Comment 3

[Mike Connor [:mconnor]]

I want to clarify one piece (r+sr) as a last round, but we're otherwise good to go real soon now.

Comment 4

[Mike Connor [:mconnor]]

Attachment: policy updates

We should separate the policy from the rules and tips/seeking review, I'd like to move those out of the document as a second step.

This is good to go, pending silent assent on the r+sr clarification in dev.planning.

Comment 5

[Mike Connor [:mconnor]]

No further feedback, the policy changes are pushed to CVS.

Comment 6

[David Boswell]

Should the page have a title instead of leading with "Introduction"?  Perhaps something like "Code Review Policy"?

Comment 7

[Gervase Markham [:gerv]]

Even after Mike's excellent work, there are still places this document needs updating, or bits need removing elsewhere. (Mike's plan in the newsgroup listed some of them.) So we can fix that along with those things.

Gerv

Comment 8

[Mitchell Baker]

Gerv

is this still on your list?  I'm guessing whole hunks at the end can go, you may be closer to this than I.

Comment 9

[Gervase Markham [:gerv]]

mitchell: still on my list, and I agree with your conclusion. Most of that end stuff can either go entirely, or be farmed out into a separate document.

Gerv

Comment 10

[Gervase Markham [:gerv]]

OK, I've overhauled the page. Please file additional bugs for further specific requested changes, if any.

Gerv