Comment 1 for bug 206508

Revision history for this message
Tim Penhey (thumper) wrote :

We are actually working on something like this, but in a slightly different way.

We are implementing code reviews within launchpad. Code reviews will be attached to a merge proposal. Following though with you example above, the contributor proposes to merge his feature branch on mainline by creating a merge proposal for that branch targetting mainline. They'd set the status to "Needs review". Someone looks at this code and will be able to comment. A conversation will be connected to the merge proposal and viewable through launchpad. This conversation will be similar to bugs in that many people can comment and vote on the code. When the proposal has been approved, it will be signed off by someone with authority to land code on the mainline branch.