Move team to peer-review process
Bug #622814 reported by
Duncan McGreggor
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Open Input Framework |
Fix Released
|
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
To post a comment you must log in.
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.