Disable verify-test-owners.sh and make `go vet` more obvious

pull/6/head
Jeff Grafton 2016-12-21 11:42:10 -08:00
parent 1955ed614f
commit 05c09b1aa3
3 changed files with 2 additions and 1 deletions

View File

@ -26,6 +26,7 @@ EXCLUDED_CHECKS=(
"verify-linkcheck.sh" # runs in separate Jenkins job once per day due to high network usage
"verify-govet.sh" # it has a separate make vet target
"verify-staging-client-go.sh" # TODO: enable the script after 1.5 code freeze
"verify-test-owners.sh" # TODO(rmmh): figure out how to avoid endless conflicts
)
function is-excluded {

View File

@ -46,4 +46,5 @@ if [[ ${#targets[@]} -eq 0 ]]; then
targets=$(go list -e ./... | egrep -v "/(third_party|vendor|staging|clientset_generated)/")
fi
set -x
go vet "${goflags[@]:+${goflags[@]}}" ${targets[@]}

View File

@ -24,6 +24,5 @@ source "${KUBE_ROOT}/hack/lib/init.sh"
cd "${KUBE_ROOT}"
if ! hack/update_owners.py --check; then
echo 'Run ./hack/update_owners.py to fix it'
exit # TODO(rmmh): fix Github merging to respect .gitattributes
exit 1
fi