Merge pull request #4 from fastbill/change-linter

change linter

.golangci.yml

@@ -0,0 +1,21 @@
+linters:
+  enable:
+    - gocyclo
+    - golint
+    - dupl
+    - interfacer
+    - unconvert
+    - goconst
+    - gosec
+    - bodyclose
+
+run:
+  deadline: 10m
+
+issues:
+  exclude-rules:
+    - path: _test\.go
+      linters:
+        - dupl
+        - goconst
+        - gosec

.travis.yml

@@ -3,15 +3,14 @@ language: go
 go:
   - 1.11.x
 
-before_install:
+install:
   - go get -u github.com/golang/dep/cmd/dep
   - dep ensure
-  - go get -u gopkg.in/alecthomas/gometalinter.v2
-  - gometalinter.v2 --install
+  - curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.17.1
 
 script:
   - go test -race -cover ./...
-  - gometalinter.v2 --vendor --deadline=300s ./...
+  - ./bin/golangci-lint run
 
 notifications:
   email: false

httperrors_test.go

@@ -63,7 +63,8 @@ func TestWriteJSON(t *testing.T) {
 		t.Run(testCase.name, func(t *testing.T) {
 			he := New(testCase.code, testCase.message)
 			w := httptest.NewRecorder()
-			he.WriteJSON(w)
+			err := he.WriteJSON(w)
+			assert.NoError(t, err)
 
 			assert.Equal(t, testCase.code, w.Code)
 			assert.Equal(t, testCase.result+"\n", w.Body.String())