Gerrit Code Review

Real World Users

• Android
• Chromium
• MediaWiki
• Qt
• OpenStack

Demo

Not perfect

• No single page diff with all files
• Little awareness of related changes
• Humans and robots treated as equals
• Knowing what needs review
• Replacing change screen with alternative that doesn't fix these problems
• Google encodes their workflow, but resists others
• Google Web Toolkit

Vinz

• Web client replacement using Gerrit REST API
• Similar to Gertty
• Modern Javascript Application
• Something we can change to accomodate user needs

Requirements

• Replacement change screen
• Basic Feature Parity with Exisitng Screen
• Commenting
• Single page diff for all files
• Diff between patchsets
• Distinguish Humans from Robots
• Dashboard(s)
• Show users what needs review
• Track important changes
• Open Source (Apache2 License)

Nice to haves

• Replacement change screen
• Commit message editor
• Bug tracker integration
• Interdiff (diff between diffs)
• User editable dashboards
• Whatever you find is useful while working the project

Questions

Thanks!

These slides are available at: http://docs.openstack.org/infra/publications/