An Overview of the Software Engineering Process in the Mozilla Project
http://www.async.com.br/~kiko/mozse/br2002/
Christian Reis
kiko@async.com.br
Renata Pontin
renata@icmc.sc.usp.br
Back | 12 | Next

Changes: code reviews

  • Review means "see if this code looks good, and if possible test it"
  • Results usually in requests to change code towards: portability, performance, legibility, completeness
  • Core modules require super review: second-level review by senior developer
  • Review, especially super review has helped improve code quality and keep regressions under control.
  • Bugzilla provides attachstatuses to support review, and special UI.
  • Over to QA!