Better module/pkg support. Argument support for all hooks. Re-worked docs

Author: TekWizely

.pre-commit-hooks.yaml

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+export GO111MODULE=on
+
 # Walks up the file path looking for go.mod
 #
 function find_module_roots() {
@@ -13,14 +15,33 @@ function find_module_roots() {
 	  while [ "${path}" != "." ] && [ ! -f "${path}/go.mod" ]; do
 	    path=$(dirname "${path}")
 	  done
-	  echo "${path}"
+	  if [ -f "${path}/go.mod" ]; then
+	  	echo "${path}"
+	  fi
 	done
 }
 
+FILES=()
+
+# Build file list while looking for optional argument marker
+#
+while [ $# -gt 0 ] && [ "$1" != "-" ] && [ "$1" != "--" ]; do
+	FILES+=("$1")
+	shift
+done
+
+# Remove argument marker if present
+#
+if [ $# -gt 0 ]; then
+	if [ "$1" == "-" ] || [ "$1" == "--" ]; then
+		shift
+	fi
+fi
+
 errCode=0
-for sub in $(find_module_roots "$@" | sort -u) ; do
+for sub in $(find_module_roots "${FILES}" | sort -u) ; do
 	pushd "${sub}" >/dev/null
-	go build ./...
+	go build "$@" ./...
 	if [ $? -ne 0 ]; then
 		errCode=1
 	fi

README.md

@@ -1,8 +1,27 @@
 #!/usr/bin/env bash
+
+export GO111MODULE=off
+
+FILES=()
+
+# Build file list while looking for optional argument marker
+#
+while [ $# -gt 0 ] && [ "$1" != "-" ] && [ "$1" != "--" ]; do
+	FILES+=("$1")
+	shift
+done
+
+# Remove argument marker if present
+#
+if [ $# -gt 0 ]; then
+	if [ "$1" == "-" ] || [ "$1" == "--" ]; then
+		shift
+	fi
+fi
+
 errCode=0
-pkg=$(go list) # repo root package
-for sub in $(echo "$@" | xargs -n1 dirname | sort -u); do
-	go build "${pkg}/${sub}"
+for sub in $(echo "${FILES}" | xargs -n1 dirname | sort -u); do
+	go build "$@" "./${sub}"
 	if [ $? -ne 0 ]; then
 		errCode=1
 	fi

go-build-dir.sh

@@ -0,0 +1,25 @@
+#!/usr/bin/env bash
+
+export GO111MODULE=on
+
+# '-' and '--' are optional argument markers for parity with file scripts
+# Remove them if present
+#
+if [ $# -gt 0 ]; then
+	if [ "$1" == "-" ] || [ "$1" == "--" ]; then
+		shift
+	fi
+fi
+
+errCode=0
+# Assume parent folder of go.mod is module root folder
+#
+for sub in $(find . -name go.mod | xargs -n1 dirname | sort -u) ; do
+	pushd "${sub}" >/dev/null
+	go build "$@" ./...
+	if [ $? -ne 0 ]; then
+		errCode=1
+	fi
+	popd >/dev/null
+done
+exit $errCode

go-build-mod.sh

@@ -1,4 +1,15 @@
 #!/usr/bin/env bash
 set -e
-pkg=$(go list) # repo root package
-go build "$@" "${pkg}/..."
+
+export GO111MODULE=off
+
+# '-' and '--' are optional argument markers for parity with file scripts
+# Remove them if present
+#
+if [ $# -gt 0 ]; then
+	if [ "$1" == "-" ] || [ "$1" == "--" ]; then
+		shift
+	fi
+fi
+
+go build "$@" ./...

go-build-pkg.sh

@@ -1,7 +1,25 @@
 #!/usr/bin/env bash
+
+FILES=()
+
+# Build file list while looking for optional argument marker
+#
+while [ $# -gt 0 ] && [ "$1" != "-" ] && [ "$1" != "--" ]; do
+	FILES+=("$1")
+	shift
+done
+
+# Remove argument marker if present
+#
+if [ $# -gt 0 ]; then
+	if [ "$1" == "-" ] || [ "$1" == "--" ]; then
+		shift
+	fi
+fi
+
 errCode=0
-for file in "$@"; do
-	gocritic check "${file}"
+for file in "${FILES}"; do
+	gocritic check "$@" "${file}"
 	if [ $? -ne 0 ]; then
 		errCode=1
 	fi