Difference between revisions of "Poppy/Developer - Review Guidelines"
< Poppy
(Created page with "* Docstrings are updated if needed * Commit messages reflect bps, bugs if applicable (https://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes)") |
|||
Line 1: | Line 1: | ||
* Docstrings are updated if needed | * Docstrings are updated if needed | ||
* Commit messages reflect bps, bugs if applicable (https://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes) | * 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 | ||
+ | YAGNI | ||
+ | DRY |
Revision as of 20:14, 10 November 2014
- 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
YAGNI DRY