Revision as of 09:51, 23 January 2015

Horizon Code Reviews

This page shares topics related to code reviews related to Horizon (including django-openstack-auth, tuskar-ui).

Please add good references useful for reviews.

• OpenStack guidelines for code reviewers
• OpenStack Style Guidelines
• i18n
• Javascript guidelines (in progress)
• Git Commit Messages
• About commits into a stable brunch (backports)

Review Links

• Direct Links
  • Horizon
  • django-openstack-auth
  • tuskar-ui

• Horizon + django-openstack-auth Review Dashboard
  • Gerrit Dashboard Creator is useful.
  • Dashboard #1
    • Generated from Created by https://github.com/stackforge/gerrit-dash-creator/tree/master/dashboards
  • Another Review Dashboard (which covers Horizon, django-openstack-auth and tuskar-ui)
    • Generated from the dashboard definition below:

 [dashboard]
 title = Horizon Review Inbox
 description = Review Inbox
 foreach = (project:openstack/horizon OR project:openstack/django_openstack_auth OR project:openstack/tuskar-ui) status:open NOT owner:self NOT label:Workflow<=-1 label:Verified>=1,jenkins NOT label:Code-Review>=0,self
 
 [section "Needs Feedback (Changes older than 5 days that have not been reviewed by anyone)"]
 query = NOT label:Code-Review<=2 age:5d
 
 [section "Your are a reviewer, but haven't voted in the current revision"]
 query = NOT label:Code-Review<=2,self reviewer:self
 
 [section "Needs final +2"]
 query = label:Code-Review>=2 limit:50
 
 [section "Passed Jenkins, No Negative Feedback"]
 query = NOT label:Code-Review>=2 NOT label:Code-Review<=-1 limit:50
 
 [section "Wayward Changes (Changes with no code review in the last 2days)"]
 query = NOT label:Code-Review<=2 age:2d