Move team to peer-review process

Bug #622814 reported by Duncan McGreggor on 2010-08-23
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Open Input Framework
High
Duncan McGreggor

Bug Description

Here are some guiding principles that need to be adopted by the uTouch development team:
 * all features needs defined tasks/bugs/tickets
 * all development tasks (bugs in Launchpad) need branches associated with them
 * no branch is merged to trunk until it has been reviewed by someone else on the team
 * no branch receives a "+1" for merge until all review feedback has been addressed to the mutual consent of developer and reviewer
 * all reviewers execute the projects test runner using the developed branch
 * no branch is approved for merge if unit tests are failing

Duncan McGreggor (oubiwann) wrote :

We have made initial strides in this regard to good effect. We're starting to see some nice results of more eyes on the code. Once we have the TDD process in place (see Bug #622807 and Bug #622809), we'll be able to fully adopt the peer-review process and close this ticket.

Duncan McGreggor (oubiwann) wrote :

Every ticket with new code is now under-going peer review.

Great job, everyone!

Changed in utouch:
milestone: none → ubuntu-10.10
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers