openstack/deb-cinder
Code Issues Proposed changes

Updated HACKING.rst so that it is accurate

Browse Source 
We haven't been very good at enforcing putting new hacking checks
into HACKING.rst.  This change brings the file up to date.

Change-Id: I95ef533d031d333da5615ead997b54c6506c2c2a
This commit is contained in:
Jay S. Bryant 2014-08-03 12:46:45 -05:00
parent b07ba26e8b
commit 79b6b98f21
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
HACKING.rst
View File

@ -7,11 +7,13 @@ Cinder Style Commandments
Cinder Specific Commandments Cinder Specific Commandments
---------------------------- ----------------------------
- [N314] Check for vi editor configuration in source files.
- [N319] Validate that debug level logs are not translated - [N319] Validate that debug level logs are not translated
- [N322] Ensure default arguments are not mutable.
- [N323] Add check for explicit import of _() to ensure proper translation. - [N323] Add check for explicit import of _() to ensure proper translation.
- [N324] Enforce no use of LOG.audit messages. LOG.info should be used instead. - [N324] Enforce no use of LOG.audit messages. LOG.info should be used instead.
General General
------- -------
- Use 'raise' instead of 'raise e' to preserve original traceback or exception being reraised:: - Use 'raise' instead of 'raise e' to preserve original traceback or exception being reraised::