Jump to: navigation, search

Difference between revisions of "Poppy/Developer - Review Guidelines"

(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

    YAGNI
    DRY