summaryrefslogtreecommitdiff
path: root/tools/run-pep8
diff options
context:
space:
mode:
authorharlowja <harlowja@virtualbox.rhel>2012-06-23 14:59:16 -0700
committerharlowja <harlowja@virtualbox.rhel>2012-06-23 14:59:16 -0700
commit67232af3cba2c7bc99c2ca67b83470b38d6db103 (patch)
tree74dbbaeafc103ff6449a0d61705c644fa6a2d838 /tools/run-pep8
parent9d1f042f862c114f1613dcd5d2d8c401a1c54eaa (diff)
downloadvyos-cloud-init-67232af3cba2c7bc99c2ca67b83470b38d6db103.tar.gz
vyos-cloud-init-67232af3cba2c7bc99c2ca67b83470b38d6db103.zip
1. Separate the pep8 check from the pylint check
a. This allows them to be run as different tools (if desired) 2. Adjust the makefile to have a 'make pep8' section which can run this new script
Diffstat (limited to 'tools/run-pep8')
-rwxr-xr-xtools/run-pep828
1 files changed, 28 insertions, 0 deletions
diff --git a/tools/run-pep8 b/tools/run-pep8
new file mode 100755
index 00000000..e7707985
--- /dev/null
+++ b/tools/run-pep8
@@ -0,0 +1,28 @@
+#!/bin/bash
+
+ci_files='cloud*.py cloudinit/*.py cloudinit/config/*.py'
+test_files=$(find tests -name "*.py")
+def_files="$ci_files $test_files"
+
+if [ $# -eq 0 ]; then
+ files=( )
+ for f in $def_files; do
+ [ -f "$f" ] || { echo "failed, $f not a file" 1>&2; exit 1; }
+ files[${#files[@]}]=${f}
+ done
+else
+ files=( "$@" );
+fi
+
+cmd=(
+ pep8
+
+ --ignore=E501 # Line too long (these are caught by pylint)
+
+ "${files[@]}"
+)
+
+echo -e "\nRunning pep8:"
+echo "${cmd[@]}"
+"${cmd[@]}"
+