diff options
author | harlowja <harlowja@virtualbox.rhel> | 2012-06-23 14:59:16 -0700 |
---|---|---|
committer | harlowja <harlowja@virtualbox.rhel> | 2012-06-23 14:59:16 -0700 |
commit | 67232af3cba2c7bc99c2ca67b83470b38d6db103 (patch) | |
tree | 74dbbaeafc103ff6449a0d61705c644fa6a2d838 /tools | |
parent | 9d1f042f862c114f1613dcd5d2d8c401a1c54eaa (diff) | |
download | vyos-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')
-rwxr-xr-x | tools/run-pep8 | 28 |
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[@]}" + |