Slight changes to codecheck utility
- Style fixes
- Supress some linting warnings
- Add project root to path
Change-Id: Ic533b939704194752df9f41a77a8e2264b6fa76f
diff --git a/TestON/bin/.pylint b/TestON/bin/.pylint
index 469a744..57208b9 100644
--- a/TestON/bin/.pylint
+++ b/TestON/bin/.pylint
@@ -13,7 +13,7 @@
# Python code to execute, usually for sys.path manipulation such as
# pygtk.require().
-#init-hook=
+init-hook='import sys;sys.path.append("/home/sdn/TestON")'
# Profiled execution.
profile=no
@@ -136,7 +136,7 @@
bad-names=foo,bar,baz,toto,tutu,tata
# List of builtins function names that should not be used, separated by a comma
-bad-functions=map,filter,apply,inpu
+bad-functions=map,filter,apply,inpu,in,list,str
# try to find bugs in the code using type inference
@@ -244,7 +244,7 @@
# Create a graph of every (i.e. internal and external) dependencies in the
# given file (report R0402 must not be disabled)
-import-graph=
+import-graph=y
# Create a graph of external dependencies in the given file (report R0402 mus
# not be disabled)
@@ -281,7 +281,7 @@
[MISCELLANEOUS]
# List of note tags to take in consideration, separated by a comma.
-notes=FIXME,XXX,TODO
+notes=FIXME,XXX,TODO,NOTE
# checks for similarities and duplicated code. This computation may be
diff --git a/TestON/bin/codecheck b/TestON/bin/codecheck
index e3769da..51b9649 100755
--- a/TestON/bin/codecheck
+++ b/TestON/bin/codecheck
@@ -30,16 +30,16 @@
autopep8 --in-place -a -a $file --ignore=$EXCLUDE,$P8IGN
elif [ -z "$2" ] || [ "$2" = "--reason" ] || [ "$2" = "--relaxed" ]; then
echo "Running pyflakes..."
- #Pyflakes is a source checker. It doesn't run the code so it is safer than other programs
- #ignoring some errors due to TestON
- pyflakes $1 | grep -v "undefined name 'main'" | grep -v "undefined name 'utilities'"
+ # Pyflakes is a source checker. It doesn't run the code so it is safer than other programs
+ # ignoring some errors due to TestON
+ pyflakes $1 | grep -v "undefined name 'main'" | grep -v "undefined name 'utilities'" | grep -v "unable to detect undefined names"
echo "Running pylint..."
- #Pylint is a static code checker
- #ignoring some errors due to TestON
+ # Pylint is a static code checker
+ # ignoring some errors due to TestON
pylint -E --rcfile=$(dirname "${BASH_SOURCE}")/.pylint $1 | grep -v "Undefined variable 'main'" | grep -v "Undefined variable 'utilities'"
echo "Running PEP8..."
- #PEP8 is a the most common python coding style standard
+ # PEP8 is a the most common python coding style standard
if [ "$2" = "--reason" ]; then
pep8 --repeat --show-source --show-pep8 --ignore=$P8IGN $1
elif [ "$2" = "--relaxed" ]; then