Services/Process/Code Review

Code Review Policy for Mozilla Services Code

Code review prior to landing is now required for all changes to all production-oriented code.

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

• Desktop core sync code
• Additional Sync code for Firefox integration

Application Services

• Application Services (shared Rust components library)

Sync storage server

• Sync storage server (in Rust)

Mozilla/Firefox Accounts

• Mozilla Accounts (formerly Firefox accounts) monorepo
• Desktop Firefox Accounts specific code