Adds github fmt, lint, test ci

coder · cmoog · Jul 29, 2020 · Jul 28, 2020 · Jul 28, 2020 · Jul 28, 2020
commit f11304ba32c6d27f9d0caf875e48163890440a92
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -9,7 +9,7 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v1
       - name: Build
-        run: ./ci/build.sh
+        run: ./ci/steps/build.sh
       - name: Upload
         uses: actions/upload-artifact@v2
         with:

diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -0,0 +1,46 @@
+name: ci
+on: [push, pull_request]
+
+jobs:
+  fmt:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - uses: actions/cache@v1
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+      - name: fmt
+        uses: ./ci/image
+        with:
+          args: ./ci/steps/fmt.sh
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - uses: actions/cache@v1
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+      - name: lint
+        uses: ./ci/image
+        with:
+          args: ./ci/steps/lint.sh
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - uses: actions/cache@v1
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+      - name: test
+        uses: ./ci/image
+        with:
+          args: go test ./...
diff --git a/ci/image/Dockerfile b/ci/image/Dockerfile
@@ -0,0 +1,8 @@
+FROM golang:1
+
+ENV GOFLAGS="-mod=readonly"
+ENV CI=true
+
+RUN go get golang.org/x/tools/cmd/goimports
+RUN go get golang.org/x/lint/golint
+RUN go get github.com/mattn/goveralls
diff --git a/ci/build.sh → ci/steps/build.sh b/ci/build.sh → ci/steps/build.sh
@@ -14,14 +14,14 @@ mkdir -p bin
 
 build(){
 	tmpdir=$(mktemp -d)
-	go build -ldflags "-s -w -X main.version=${tag}" -o "$tmpdir/coder" ../cmd/coder
+	go build -ldflags "-s -w -X main.version=${tag}" -o "$tmpdir/coder" ../../cmd/coder
 
 	pushd "$tmpdir"
 		tarname="coder-cli-$GOOS-$GOARCH.tar.gz"
 		tar -czf "$tarname" coder
 	popd
 
-	cp "$tmpdir/$tarname" bin
+	cp "$tmpdir/$tarname" ../bin
 	rm -rf "$tmpdir"
 }
 

diff --git a/ci/steps/fmt.sh b/ci/steps/fmt.sh
@@ -0,0 +1,16 @@
+#!/bin/bash
+echo "Formatting..."
+
+go mod tidy
+gofmt -w -s .
+goimports -w "-local=$$(go list -m)" .
+
+if [ "$CI" != "" ]; then
+  if [[ $(git ls-files --other --modified --exclude-standard) != "" ]]; then
+    echo "Files need generation or are formatted incorrectly:"
+    git -c color.ui=always status | grep --color=no '\e\[31m'
+    echo "Please run the following locally:"
+    echo "  ./ci/fmt.sh"
+    exit 1
+  fi
+fi
diff --git a/ci/steps/lint.sh b/ci/steps/lint.sh
@@ -0,0 +1,6 @@
+#!/bin/bash
+
+echo "Linting..."
+
+go vet ./...
+golint -set_exit_status ./...