Add gofmt script

Signed-off-by: jwcesign <[email protected]>

.golangci.yml

@@ -13,10 +13,12 @@ run:
   # on Windows.
   skip-dirs:
   - hack/tools/preferredimports # This code is directly lifted from the Kubernetes codebase, skip checking
+  - (^|/)vendor($|/)
+  - (^|/)third_party($|/)
 
   # default is true. Enables skipping of directories:
   #   vendor$, third_party$, testdata$, examples$, Godeps$, builtin$
-  skip-dirs-use-default: true
+  skip-dirs-use-default: false
 
   # One of 'readonly' and 'vendor'.
   #  - readonly: the go command is disallowed from the implicit automatic updating of go.mod described above.

examples/customresourceinterpreter/webhook/app/workloadwebhook.go

@@ -139,7 +139,7 @@ func (e *workloadInterpreter) responseWithExploreInterpretStatus(workload *workl
 
 	res := interpreter.Succeeded("")
 	res.RawStatus = &runtime.RawExtension{
-		Raw:    marshaledBytes,
+		Raw: marshaledBytes,
 	}
 
 	return res

hack/update-all.sh

@@ -15,4 +15,4 @@ bash "$REPO_ROOT/hack/update-import-aliases.sh"
 bash "$REPO_ROOT/hack/update-swagger-docs.sh"
 bash "$REPO_ROOT/hack/update-lifted.sh"
 bash "$REPO_ROOT/hack/update-mocks.sh"
-
+bash "$REPO_ROOT/hack/update-gofmt.sh"

hack/update-gofmt.sh

@@ -0,0 +1,42 @@
+#!/usr/bin/env bash
+
+# Copyright 2014 The Kubernetes Authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+# GoFmt apparently is changing @ head...
+
+set -o errexit
+set -o nounset
+set -o pipefail
+
+REPO_ROOT=$(dirname "${BASH_SOURCE[0]}")/..
+source "${REPO_ROOT}"/hack/util.sh
+
+util::verify_go_version
+
+cd "${REPO_ROOT}"
+
+find_files() {
+  find . -not \( \
+      \( \
+        -wholename './output' \
+        -o -wholename './.git' \
+        -o -wholename './_output' \
+        -o -wholename '*/third_party/*' \
+        -o -wholename '*/vendor/*' \
+      \) -prune \
+    \) -name '*.go'
+}
+
+find_files | xargs gofmt -s -w

hack/verify-all.sh

@@ -16,6 +16,7 @@ bash "$REPO_ROOT/hack/verify-lifted.sh"
 bash "$REPO_ROOT/hack/verify-import-aliases.sh"
 bash "$REPO_ROOT/hack/verify-staticcheck.sh"
 bash "$REPO_ROOT/hack/verify-mocks.sh"
+bash "$REPO_ROOT/hack/verify-gofmt.sh"
 bash "$REPO_ROOT/hack/verify-vendor.sh"
 bash "$REPO_ROOT/hack/verify-swagger-docs.sh"
 bash "$REPO_ROOT/hack/verify-crdgen.sh"

hack/verify-gofmt.sh

@@ -0,0 +1,57 @@
+#!/usr/bin/env bash
+
+# Copyright 2014 The Kubernetes Authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+# This script checks whether the source code needs to be formatted or not by
+# `gofmt`. Run `hack/update-gofmt.sh` to actually format sources.
+#
+# Note: gofmt output can change between go versions.
+#
+# Usage: `hack/verify-gofmt.sh`.
+
+set -o errexit
+set -o nounset
+set -o pipefail
+
+REPO_ROOT=$(dirname "${BASH_SOURCE[0]}")/..
+source "${REPO_ROOT}"/hack/util.sh
+
+util::verify_go_version
+
+cd "${REPO_ROOT}"
+
+find_files() {
+  find . -not \( \
+      \( \
+        -wholename './output' \
+        -o -wholename './.git' \
+        -o -wholename './_output' \
+        -o -wholename '*/third_party/*' \
+        -o -wholename '*/vendor/*' \
+      \) -prune \
+    \) -name '*.go'
+}
+
+# gofmt exits with non-zero exit code if it finds a problem unrelated to
+# formatting (e.g., a file does not parse correctly). Without "|| true" this
+# would have led to no useful error message from gofmt, because the script would
+# have failed before getting to the "echo" in the block below.
+diff=$(find_files | xargs gofmt -d -s 2>&1) || true
+if [[ -n "${diff}" ]]; then
+  echo "${diff}" >&2
+  echo >&2
+  echo "Run ./hack/update-gofmt.sh" >&2
+  exit 1
+fi