Add gofmt -s to CI (#146)

* Add gofmt to CI

* Apply gofmt -s

* Add check job to CI

* Install go-fmt-fail

* Revert "Add check job to CI"

This reverts commit f923bc2.

* Fix Makefile fmt command

* Add .gitattributes

.gitattributes

@@ -0,0 +1 @@
+*.go text eol=lf

.github/workflows/ci.yml

@@ -53,6 +53,9 @@ jobs:
       -
         name: Go mod verify
         run: go mod verify
+      -
+        name: Run go fmt
+        run: make fmt
       -
         name: Run tests
         run: make test

Makefile

@@ -1,6 +1,11 @@
+export GOFLAGS = -mod=vendor
+
 default: test
 
+fmt:
+	go run github.com/mh-cbon/go-fmt-fail ./...
+
 test:
-	go test -mod=vendor -v -cover ./...
+	go test -v -cover ./...
 
-.PHONY: test
+.PHONY: fmt test

go.mod

@@ -11,6 +11,7 @@ require (
 	github.com/hashicorp/go-version v1.2.0
 	github.com/hashicorp/hcl/v2 v2.5.2-0.20200528183353-fa7c453538de
 	github.com/hashicorp/terraform-json v0.5.0
+	github.com/mh-cbon/go-fmt-fail v0.0.0-20160815164508-67765b3fbcb5
 	github.com/mitchellh/cli v1.0.0
 	github.com/pmezard/go-difflib v1.0.0
 	github.com/sourcegraph/go-lsp v0.0.0-20200117082640-b19bb38222e2

go.sum

@@ -51,6 +51,8 @@ github.com/mattn/go-colorable v0.0.9 h1:UVL0vNpWh04HeJXV0KLcaT7r06gOH2l4OW6ddYRU
 github.com/mattn/go-colorable v0.0.9/go.mod h1:9vuHe8Xs5qXnSaW/c/ABM9alt+Vo+STaOChaDxuIBZU=
 github.com/mattn/go-isatty v0.0.3 h1:ns/ykhmWi7G9O+8a448SecJU3nSMBXJfqQkl0upE1jI=
 github.com/mattn/go-isatty v0.0.3/go.mod h1:M+lRXTBqGeGNdLjl/ufCoiOlB5xdOkqRJdNxMWT7Zi4=
+github.com/mh-cbon/go-fmt-fail v0.0.0-20160815164508-67765b3fbcb5 h1:shw+DWUaHIyW64Tv30ASCbC6QO6fLy+M5SJb5pJVEI4=
+github.com/mh-cbon/go-fmt-fail v0.0.0-20160815164508-67765b3fbcb5/go.mod h1:nHPoxaBUc5CDAMIv0MNmn5PBjWbTs9BI/eh30/n0U6g=
 github.com/mitchellh/cli v1.0.0 h1:iGBIsUe3+HZ/AD/Vd7DErOt5sU9fa8Uj7A2s1aggv1Y=
 github.com/mitchellh/cli v1.0.0/go.mod h1:hNIlj7HEI86fIcpObd7a0FcrxTWetlwJDGcceTlRvqc=
 github.com/mitchellh/go-wordwrap v0.0.0-20150314170334-ad45545899c7 h1:DpOJ2HYzCv8LZP15IdmG+YdwD2luVPHITV96TkirNBM=

internal/terraform/lang/config_block_test.go

@@ -301,7 +301,7 @@ func TestCompletableLabels_CompletionCandidatesAtPos_overLimit(t *testing.T) {
 		},
 		tBlock: tBlock,
 		labels: map[string][]*labelCandidate{
-			"type": []*labelCandidate{
+			"type": {
 				{label: "aaa"},
 				{label: "bbb"},
 				{label: "ccc"},
@@ -338,7 +338,7 @@ func TestCompletableLabels_CompletionCandidatesAtPos_matchingLimit(t *testing.T)
 		},
 		tBlock: tBlock,
 		labels: map[string][]*labelCandidate{
-			"type": []*labelCandidate{
+			"type": {
 				{label: "aaa"},
 				{label: "bbb"},
 			},
@@ -374,7 +374,7 @@ func TestCompletableLabels_CompletionCandidatesAtPos_withPrefix(t *testing.T) {
 		},
 		tBlock: tBlock,
 		labels: map[string][]*labelCandidate{
-			"type": []*labelCandidate{
+			"type": {
 				{label: "prov_aaa"},
 				{label: "prov_bbb"},
 				{label: "ccc"},

tools.go

@@ -0,0 +1,5 @@
+// +build tools
+
+package main
+
+import _ "github.com/mh-cbon/go-fmt-fail"