Jump to: navigation, search

ReviewChecklist

Revision as of 22:04, 2 May 2013 by Beagles (talk | contribs)

This page includes a list of things that reviewers should keep in mind when reviewing patches to OpenStack projects. There are some items that are common across all projects and others that are specific to a project.

Common Review Checklist

1. The code should comply with everything in HACKING.

2. When adding a new file:

  • a) If it's not installed by setup.py, but should be included in the tarball, be sure to add it to `MANIFEST.in`.

Nova Review Checklist

1. When adding a new file:

  1. Check common checklist items for adding new files.
  2. If you are adding a new script in the `bin/` directory, add it to `setup.py`.

Notes for Non-Core Developers

  • When you are reviewing, you may notice a review that has several +1's from other reviewers, passes the functional tests, etc. but the code still has not been merged. As only core developers can approve code for merging, you can help things along by getting a core developer's attention and letting them know there is a review with lots of positive reviews and needs final approval.