coder · mafredri · May 27, 2022 · May 23, 2022 · May 23, 2022 · May 23, 2022
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,16 @@
+root = true
+
+[*]
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+indent_style = tab
+
+[*.{md,json,yaml,tf,tfvars}]
+indent_style = space
+indent_size = 2
+
+[coderd/database/dump.sql]
+indent_style = space
+indent_size = 4
diff --git a/Makefile b/Makefile
@@ -47,6 +47,7 @@ fmt: fmt/prettier fmt/terraform
 gen: coderd/database/querier.go peerbroker/proto/peerbroker.pb.go provisionersdk/proto/provisioner.pb.go provisionerd/proto/provisionerd.pb.go site/src/api/typesGenerated.ts
 
 install: build
+	mkdir -p $(INSTALL_DIR)
 	@echo "--- Copying from bin to $(INSTALL_DIR)"
 	cp -r ./dist/coder-$(GOOS)_$(GOOS)_$(GOARCH)*/* $(INSTALL_DIR)
 	@echo "-- CLI available at $(shell ls $(INSTALL_DIR)/coder*)"

diff --git a/coderd/audit/generate.sh b/coderd/audit/generate.sh
@@ -8,8 +8,12 @@
 # Usage:
 # ./generate.sh <database type> <database type> ...
 
-
 set -euo pipefail
 
-cd "$(dirname "$0")" && cd "$(git rev-parse --show-toplevel)"
-go run ./scripts/auditgen ./coderd/database "$@"
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+PROJECT_ROOT=$(cd "$SCRIPT_DIR" && git rev-parse --show-toplevel)
+
+(
+	cd "$PROJECT_ROOT"
+	go run ./scripts/auditgen ./coderd/database "$@"
+)
diff --git a/coderd/database/generate.sh b/coderd/database/generate.sh
@@ -8,39 +8,43 @@
 
 set -euo pipefail
 
-cd "$(dirname "$0")"
-
-# The logic below depends on the exact version being correct :(
-go run github.com/kyleconroy/sqlc/cmd/sqlc@v1.13.0 generate
-
-first=true
-for fi in queries/*.sql.go; do
-    # Find the last line from the imports section and add 1.
-    cut=$(grep -n ')' "$fi" | head -n 1 | cut -d: -f1)
-    cut=$((cut + 1))
-
-    # Copy the header from the first file only, ignoring the source comment.
-    if $first; then
-        head -n 6 < "$fi" | grep -v "source" > queries.sql.go
-        first=false
-    fi
-
-    # Append the file past the imports section into queries.sql.go.
-    tail -n "+$cut" < "$fi" >> queries.sql.go
-done
-
-# Move the files we want.
-mv queries/querier.go .
-mv queries/models.go .
-
-# Remove temporary go files.
-rm -f queries/*.go
-
-# Fix struct/interface names.
-gofmt -w -r 'Querier -> querier' -- *.go
-gofmt -w -r 'Queries -> sqlQuerier' -- *.go
-
-# Ensure correct imports exist. Modules must all be downloaded so we get correct
-# suggestions.
-go mod download
-goimports -w queries.sql.go
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+
+(
+	cd "$SCRIPT_DIR"
+
+	# The logic below depends on the exact version being correct :(
+	go run github.com/kyleconroy/sqlc/cmd/sqlc@v1.13.0 generate
+
+	first=true
+	for fi in queries/*.sql.go; do
+		# Find the last line from the imports section and add 1.
+		cut=$(grep -n ')' "$fi" | head -n 1 | cut -d: -f1)
+		cut=$((cut + 1))
+
+		# Copy the header from the first file only, ignoring the source comment.
+		if $first; then
+			head -n 6 <"$fi" | grep -v "source" >queries.sql.go
+			first=false
+		fi
+
+		# Append the file past the imports section into queries.sql.go.
+		tail -n "+$cut" <"$fi" >>queries.sql.go
+	done
+
+	# Move the files we want.
+	mv queries/querier.go .
+	mv queries/models.go .
+
+	# Remove temporary go files.
+	rm -f queries/*.go
+
+	# Fix struct/interface names.
+	gofmt -w -r 'Querier -> querier' -- *.go
+	gofmt -w -r 'Queries -> sqlQuerier' -- *.go
+
+	# Ensure correct imports exist. Modules must all be downloaded so we get correct
+	# suggestions.
+	go mod download
+	goimports -w queries.sql.go
+)
diff --git a/coderd/database/migrations/create_migration.sh b/coderd/database/migrations/create_migration.sh
@@ -7,14 +7,17 @@
 
 set -euo pipefail
 
-cd "$(dirname "$0")"
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+(
+	cd "$SCRIPT_DIR"
 
-# if migration name is an empty string exit
-[[ -z "${*}" ]] && (echo "Must provide a migration name" && exit 1)
+	# if migration name is an empty string exit
+	[[ -z "${*}" ]] && (echo "Must provide a migration name" && exit 1)
 
-# " " && "-" -> "_"
-title="$(echo "${@}" | tr "[:upper:]" "[:lower:]" | sed -E -e "s/( |-)/_/g")"
+	# " " && "-" -> "_"
+	title="$(echo "${@}" | tr "[:upper:]" "[:lower:]" | sed -E -e "s/( |-)/_/g")"
 
-migrate create -ext sql -dir . -seq "$title"
+	migrate create -ext sql -dir . -seq "$title"
 
-echo "Run \"make gen\" to generate models."
+	echo "Run \"make gen\" to generate models."
+)
diff --git a/scripts/check_unstaged.sh b/scripts/check_unstaged.sh
@@ -2,23 +2,29 @@
 
 set -euo pipefail
 
-FILES="$(git ls-files --other --modified --exclude-standard)"
-if [[ "$FILES" != "" ]]; then
-    mapfile -t files <<<"$FILES"
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+PROJECT_ROOT=$(cd "$SCRIPT_DIR" && git rev-parse --show-toplevel)
 
-    echo "The following files contain unstaged changes:"
-    echo
-    for file in "${files[@]}"; do
-        echo "  - $file"
-    done
-    echo
+(
+	cd "${PROJECT_ROOT}"
 
-    echo "These are the changes:"
-    echo
-    for file in "${files[@]}"; do
-        git --no-pager diff "$file"
-    done
-    exit 1
-fi
+	FILES="$(git ls-files --other --modified --exclude-standard)"
+	if [[ "$FILES" != "" ]]; then
+		mapfile -t files <<<"$FILES"
 
+		echo "The following files contain unstaged changes:"
+		echo
+		for file in "${files[@]}"; do
+			echo "  - $file"
+		done
+		echo
+
+		echo "These are the changes:"
+		echo
+		for file in "${files[@]}"; do
+			git --no-pager diff "$file"
+		done
+		exit 1
+	fi
+)
 exit 0
diff --git a/scripts/develop.sh b/scripts/develop.sh
@@ -2,8 +2,8 @@
 
 set -euo pipefail
 
-PROJECT_ROOT="$(git rev-parse --show-toplevel)"
-cd "${PROJECT_ROOT}"
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+PROJECT_ROOT=$(cd "$SCRIPT_DIR" && git rev-parse --show-toplevel)
 
 echo '== Run "make build" before running this command to build binaries.'
 echo '== Without these binaries, workspaces will fail to start!'
@@ -19,8 +19,10 @@ export CODER_DEV_ADMIN_PASSWORD=password
 # to kill both at the same time. For more details, see:
 # https://stackoverflow.com/questions/3004811/how-do-you-run-multiple-programs-in-parallel-from-a-bash-script
 (
-  trap 'kill 0' SIGINT
-  CODERV2_HOST=http://127.0.0.1:3000 INSPECT_XSTATE=true yarn --cwd=./site dev &
-  go run -tags embed cmd/coder/main.go server --dev --tunnel=true &
-  wait
+	cd "${PROJECT_ROOT}"
+
+	trap 'kill 0' SIGINT
+	CODERV2_HOST=http://127.0.0.1:3000 INSPECT_XSTATE=true yarn --cwd=./site dev &
+	go run -tags embed cmd/coder/main.go server --dev --tunnel=true &
+	wait
 )
diff --git a/scripts/sign_macos.sh b/scripts/sign_macos.sh
@@ -1,17 +1,23 @@
 #!/usr/bin/env bash
 
 set -euo pipefail
-cd "$(git rev-parse --show-toplevel)"
 
-codesign -s $AC_APPLICATION_IDENTITY -f -v --timestamp --options runtime $1 
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+PROJECT_ROOT=$(cd "$SCRIPT_DIR" && git rev-parse --show-toplevel)
 
-config="$(mktemp -d)/gon.json" 
-jq -r --null-input --arg path "$(pwd)/$1" '{
-    "notarize": [
-        {
-            "path": $path,
-            "bundle_id": "com.coder.cli"
-        }
-    ]
-}' > $config
-gon $config
+(
+	cd "${PROJECT_ROOT}"
+
+	codesign -s "$AC_APPLICATION_IDENTITY" -f -v --timestamp --options runtime "$1"
+
+	config=$(mktemp -d)/gon.json
+	jq -r --null-input --arg path "$(pwd)/$1" '{
+		"notarize": [
+			{
+				"path": $path,
+				"bundle_id": "com.coder.cli"
+			}
+		]
+	}' >"$config"
+	gon "$config"
+)
diff --git a/scripts/yarn_install.sh b/scripts/yarn_install.sh
@@ -7,33 +7,37 @@
 
 set -euo pipefail
 
-PROJECT_ROOT=$(git rev-parse --show-toplevel)
-cd "$PROJECT_ROOT/site"
+SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
+PROJECT_ROOT=$(cd "$SCRIPT_DIR" && git rev-parse --show-toplevel)
 
-yarn_flags=(
-  # Do not execute install scripts
-  # TODO: check if build works properly with this enabled
-  # --ignore-scripts
+(
+	cd "$PROJECT_ROOT/site"
 
-  # Check if existing node_modules are valid
-  # TODO: determine if this is necessary
-  # --check-files
-)
+	yarn_flags=(
+		# Do not execute install scripts
+		# TODO: check if build works properly with this enabled
+		# --ignore-scripts
+
+		# Check if existing node_modules are valid
+		# TODO: determine if this is necessary
+		# --check-files
+	)
+
+	if [ -n "${CI:-}" ]; then
+		yarn_flags+=(
+			# Install dependencies from lockfile, ensuring builds are fully
+			# reproducible
+			--frozen-lockfile
+			# Suppress progress information
+			--silent
+			# Disable interactive prompts for build
+			--non-interactive
+		)
+	fi
 
-if [ -n "${CI:-}" ]; then
-  yarn_flags+=(
-    # Install dependencies from lockfile, ensuring builds are fully
-    # reproducible
-    --frozen-lockfile
-    # Suppress progress information
-    --silent
-    # Disable interactive prompts for build
-    --non-interactive
-  )
-fi
-
-# Append whatever is specified on the command line
-yarn_flags+=("$@")
-
-echo "+ yarn install ${yarn_flags[*]}"
-yarn install "${yarn_flags[@]}"
+	# Append whatever is specified on the command line
+	yarn_flags+=("$@")
+
+	echo "+ yarn install ${yarn_flags[*]}"
+	yarn install "${yarn_flags[@]}"
+)
diff --git a/site/.editorconfig b/site/.editorconfig
@@ -0,0 +1,7 @@
+[*]
+indent_style = space
+indent_size = 2
+
+[*.go]
+indent_style = tab
+indent_size = unset