Diff for "Reviews"

Not logged in - Log In / Register

Differences between revisions 1 and 11 (spanning 10 versions)
Revision 1 as of 2008-11-19 14:21:59
Size: 1291
Editor: barry
Comment:
Revision 11 as of 2009-01-14 01:57:18
Size: 964
Editor: kfogel
Comment: Make this page be just about review.
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
= Hacking Launchpad =

These pages describe processes and guidelines for developing the software that
runs Launchpad. It includes tips for coders and tips for reviewers, as well
as public meeting notes, and anything else that is useful to people actually
writing and reviewing Launchpad code. Start here if you like to get your
hands dirty.


== Developer resources ==

 * Coming soon...

== Reviewer resources ==
= Review =
Line 25: Line 12:
The list of reviewers and their on-call schedule is [[ReviewerSchedule||available here]].
Line 27: Line 16:
 * Coming soon...  * Coming soon.

Review

No changes to Launchpad can land without being reviewed and approved by at least one other person. Our goal is for every Launchpad developer to also be a reviewer, and to that end we have a mentoring process to initiate new developers onto the review team. Every "mentat" (new reviewer) is assigned a mentor (experienced reviewer) to help guide them through the process. Mentats do the reviews and the mentor reviews the mentat's review. When the mentor and review team lead agree that the mentat is ready to leave the nest, the mentat is graduated and becomes a full-fledged reviewer.

The list of reviewers and their on-call schedule is ReviewerSchedule.

Here are resources we've established to help reviewers:

  • Coming soon.

We also conduct two weekly reviewer meetings on a public IRC channel. One meeting is for Antipodeans, and the other is for Europeans and Americans.

Reviews (last edited 2020-10-06 08:16:24 by ilasc)