Poppy/Developer - Review Guidelines

Poppy Code Review Guidelines
Below are 'some' of the things to watch out for during code reviews. Please update this list as you come across new ones.


 * Docstrings are updated if needed
 * Commit messages reflect bps, bugs if applicable (https://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes)
 * No trailing whitespaces :)
 * Keep in mind the design priniciples
 * 1) YAGNI http://en.wikipedia.org/wiki/You_aren't_gonna_need_it
 * 2) DRY http://en.wikipedia.org/wiki/Don%27t_repeat_yourself