Add go lint checking for gwctl

.golangci.yml

@@ -72,5 +72,9 @@ issues:
     - linters:
       - testifylint
       text: "contains assertions that must only be used in the goroutine running the test function"
+    # It is valid usage to wrap errors without using %w to not make them part of
+    # the API contract.
+    - linters: ["errorlint"]
+      text: "non-wrapping format verb for fmt.Errorf. Use `%w` to format errors"
   exclude:
     - Using the variable on range scope `tc` in function literal

hack/verify-golint.sh

@@ -26,6 +26,25 @@ cd "${KUBE_ROOT}"
 # See configuration file in ${KUBE_ROOT}/.golangci.yml.
 mkdir -p cache
 
-docker run --rm -v $(pwd)/cache:/cache -v $(pwd):/app --security-opt="label=disable" -e GOLANGCI_LINT_CACHE=/cache -e GOFLAGS="-buildvcs=false"  -w /app "golangci/golangci-lint:$VERSION" golangci-lint run
+failed=false
+for module in $(find . -name "go.mod" | xargs -n1 dirname); do
+  echo "Linting ${module}"
+
+  docker run --rm \
+    -v $(pwd)/cache:/cache \
+    -v $(pwd):/app \
+    -w "/app/${module}" \
+    --security-opt="label=disable" \
+    -e GOLANGCI_LINT_CACHE=/cache \
+    -e GOFLAGS="-buildvcs=false" \
+    "golangci/golangci-lint:$VERSION" \
+    golangci-lint run || failed=true
+done
+
+if ${failed}; then
+  exit 1
+else
+  exit 0
+fi
 
 # ex: ts=2 sw=2 et filetype=sh