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