Services/Process/Code Review

From MozillaWiki
< Services‎ | Process(Redirected from Services/Code Review)
Jump to: navigation, search

Code Review Policy for Mozilla Services Code

Code review prior to landing is now required for _all_ changes to all production-oriented Mozilla Services code. This includes the Firefox Sync client code, the various pieces of server code, and anything else living in http://hg.mozilla.org/services.

All changes must have at least one review, and all large changes (significant refactorings, new features, new codebases) must have at least two reviews from qualified reviewers. Where it is unclear, the first reviewer should determine that a second review would be appropriate.

Code Areas

Firefox Sync

Repositories

Reviewers

Android Sync

No formal code review policy yet. :rnewman owns this one.

Firefox Share (f.k.a. F1)

Repositories

Reviewers

Identity client

Repositories

Reviewers

Server Code (PHP)

Repositories

Reviewers

Sync/Reg Server Code (Python)

Repositories

Reviewers

Key Exchange Server (Python)

Repositories

Reviewers

Identity server

Repositories

Reviewers