openstack/deb-python-hacking
Code Issues Proposed changes

Merge "fix N401 errors, stop ignoring all N4* errors"

Browse Source
This commit is contained in:
Jenkins
2013-01-09 01:01:57 +00:00
committed by Gerrit Code Review
parent 40c2fc4ebc 0a4626a7a6
commit 0e063c82f6
1 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
hacking.py
View File

@@ -268,18 +268,23 @@ def nova_import_no_db_in_virt(logical_line, filename):
yield (0, "NOVA N307: nova.db import not allowed in nova/virt/*") yield (0, "NOVA N307: nova.db import not allowed in nova/virt/*")
def nova_docstring_start_space(physical_line): def nova_docstring_start_space(physical_line, previous_logical):
"""Check for docstring not start with space. """Check for docstring not start with space.
nova HACKING guide recommendation for docstring: nova HACKING guide recommendation for docstring:
Docstring should not start with space Docstring should not start with space
N401 N401
""" """
pos = max([physical_line.find(i) for i in DOCSTRING_TRIPLE]) # start # it's important that we determine this is actually a docstring,
if (pos != -1 and len(physical_line) > pos + 1): # and not a doc block used somewhere after the first line of a
if (physical_line[pos + 3] == ' '): # function def
return (pos, "NOVA N401: one line docstring should not start with" if (previous_logical.startswith("def ") or
" a space") previous_logical.startswith("class ")):
pos = max([physical_line.find(i) for i in DOCSTRING_TRIPLE])
if (pos != -1 and len(physical_line) > pos + 4):
if (physical_line[pos + 3] == ' '):
return (pos, "NOVA N401: docstring should not start with"
" a space")
def nova_docstring_one_line(physical_line): def nova_docstring_one_line(physical_line):