For several years, there was a formal quality team that did code reviews on each commit on LTS stable branches. This was quite resource intense as a team 5 developers would review, and 3 approvals were necessary for a commit to be approved. And there was not a proper code review tool (votes were handled by the commit mailing list). Thus, this effort has ceased and instead the code review is done "wiki way". This team (as it was) will be inactive for the foreseeable future. Any quality related activities will instead be via automated tools, part of
Testing Team. The information below is kept for posterity.
The Quality Team aims to (especially in the stable branch) to share experience and minimize the risk of introducing regression bugs or bad coding practices.
Although there is currently no formal code review process for commits, it is currently being done by the
Developers Team on an ongoing basis through the monitoring of the SVN mailing list. A formal code review process has been implemented before but only for commits in the LTS branches, in the form of a Quality Team. This was abandoned when the amount of overhead became unsustainable. There has been talk about getting some kind of code review process up again, but details are uncertain - in order to have a process that works smoothly and does not depend on too small a group of developers. The plan is to wait till after we've switched to GIT from SVN as that would address a number of issues we faced before including merging issues between the various SVN branches. GIT is also a much better tool via "pull requests" to implement a more distributed code review process. See
Distributed revision control.
Release responsibilities
Ongoing responsibilities
Task
- Clean up content and move it to proper site according to Where.
Projects
More details: dev:Quality Team
Aliases