Add gometalinter to CI

Signed-off-by: Daniel Nephin <[email protected]>

Author: dnephin

Makefile

@@ -2,7 +2,7 @@
 # github.com/docker/cli 
 #
 
-.PHONY: build clean cross
+.PHONY: build clean cross test lint
 
 # build the CLI
 build: clean
@@ -17,6 +17,9 @@ clean:
 test:
 	@go test -tags daemon -v $(shell go list ./... | grep -v /vendor/)
 
+lint:
+	@gometalinter --config gometalinter.json ./...
+
 # build the CLI for multiple architectures
 cross: clean
 	@gox -output build/docker-{{.OS}}-{{.Arch}} \

circle.yml

@@ -7,4 +7,11 @@ jobs:
     steps:
       - checkout
       - setup_remote_docker
-      - run: docker build -f dockerfiles/Dockerfile.ci .
+      - run: 
+          name: "Lint"          
+          command: |
+            docker build -f dockerfiles/Dockerfile.lint --tag cli-linter .
+            docker run cli-linter
+      - run:
+          name: "Build and Unit Test"
+          command: docker build -f dockerfiles/Dockerfile.ci .

cli/command/container/exec_test.go

@@ -13,21 +13,21 @@ type arguments struct {
 
 func TestParseExec(t *testing.T) {
 	valids := map[*arguments]*types.ExecConfig{
-		&arguments{
+		{
 			execCmd: []string{"command"},
 		}: {
 			Cmd:          []string{"command"},
 			AttachStdout: true,
 			AttachStderr: true,
 		},
-		&arguments{
+		{
 			execCmd: []string{"command1", "command2"},
 		}: {
 			Cmd:          []string{"command1", "command2"},
 			AttachStdout: true,
 			AttachStderr: true,
 		},
-		&arguments{
+		{
 			options: execOptions{
 				interactive: true,
 				tty:         true,
@@ -42,7 +42,7 @@ func TestParseExec(t *testing.T) {
 			Tty:          true,
 			Cmd:          []string{"command"},
 		},
-		&arguments{
+		{
 			options: execOptions{
 				detach: true,
 			},
@@ -54,7 +54,7 @@ func TestParseExec(t *testing.T) {
 			Detach:       true,
 			Cmd:          []string{"command"},
 		},
-		&arguments{
+		{
 			options: execOptions{
 				tty:         true,
 				interactive: true,

dockerfiles/Dockerfile.lint

@@ -0,0 +1,12 @@
+FROM    golang:1.8-alpine
+
+RUN     apk add -U git
+
+RUN     go get -u gopkg.in/alecthomas/gometalinter.v1 && \
+        mv /go/bin/gometalinter.v1 /usr/local/bin/gometalinter && \
+        gometalinter --install
+
+WORKDIR /go/src/github.com/docker/cli
+COPY . .
+ENTRYPOINT ["/usr/local/bin/gometalinter"]
+CMD ["--config=gometalinter.json", "./..."]

gometalinter.json

@@ -0,0 +1,8 @@
+{
+  "Vendor": true,
+  "Sort": ["linter", "severity"],
+  "Exclude": ["cli/compose/schema/bindata.go"],
+
+  "DisableAll": true,
+  "Enable": ["gofmt", "vet"]
+}