Improved tools/hacking.py

* cleaner output
* fix bug 980009
* Fix N201
* N306: alphabetical order imports
* N401: docstring start
* N402: one line docstring start
* N403: multi line docstring end
* Until fixed, N40* will be disabled by default

Change-Id: I9addafdaa7a1f8fb950e14a5409f661dec6c7b87
This commit is contained in:
Joe Gordon 2012-04-13 16:46:42 -04:00
parent 6e2f970c1d
commit e6b9279da4
2 changed files with 5 additions and 2 deletions

View File

@ -82,7 +82,7 @@ Example::
"""A one line docstring looks like this and ends in a period.""" """A one line docstring looks like this and ends in a period."""
"""A multiline docstring has a one-line summary, less than 80 characters. """A multi line docstring has a one-line summary, less than 80 characters.
Then a new paragraph after a newline that explains in more detail any Then a new paragraph after a newline that explains in more detail any
general information about the function, class or method. Example usages general information about the function, class or method. Example usages

View File

@ -108,7 +108,10 @@ function run_pep8 {
echo "Running PEP8 and HACKING compliance check..." echo "Running PEP8 and HACKING compliance check..."
# Just run PEP8 in current environment # Just run PEP8 in current environment
# #
${wrapper} python tools/hacking.py ${srcfiles}
# Until all these issues get fixed, ignore.
ignore='--ignore=N4,N306'
${wrapper} python tools/hacking.py ${ignore} ${srcfiles}
} }