efa7068133
The validate-lint script excludes any package names that match api/types. However, the only subpackage that appears to cause issues is api/types/container (due to stuttering names). Tighten the filtering so that other code inside api/types is validated. Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
31 lines
703 B
Bash
Executable file
31 lines
703 B
Bash
Executable file
#!/bin/bash
|
|
|
|
export SCRIPTDIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
|
|
source "${SCRIPTDIR}/.validate"
|
|
|
|
IFS=$'\n'
|
|
files=( $(validate_diff --diff-filter=ACMR --name-only -- '*.go' | grep -v '^vendor/' | grep -v '^api/types/container/' || true) )
|
|
unset IFS
|
|
|
|
errors=()
|
|
for f in "${files[@]}"; do
|
|
failedLint=$(golint "$f")
|
|
if [ "$failedLint" ]; then
|
|
errors+=( "$failedLint" )
|
|
fi
|
|
done
|
|
|
|
if [ ${#errors[@]} -eq 0 ]; then
|
|
echo 'Congratulations! All Go source files have been linted.'
|
|
else
|
|
{
|
|
echo "Errors from golint:"
|
|
for err in "${errors[@]}"; do
|
|
echo "$err"
|
|
done
|
|
echo
|
|
echo 'Please fix the above errors. You can test via "golint" and commit the result.'
|
|
echo
|
|
} >&2
|
|
false
|
|
fi
|