Some small fixes (#401)

* fix golint exit status

* add comments to fix golint complaining

* make go vet verbose

* fix wrong usage of printing

.travis.yml

@@ -15,8 +15,8 @@ jobs:
     - stage: lint
       script:
         - go get github.com/golang/lint/golint
-        - golint
-        - go vet
+        - golint -set_exit_status
+        - go vet -v
     - stage: test
       script:
         - go test -v

merge_request_approvals.go

@@ -61,6 +61,10 @@ func (m MergeRequestApprovals) String() string {
 	return Stringify(m)
 }
 
+// ApproveMergeRequestOptions represents the available ApproveMergeRequest() options.
+//
+// GitLab API docs:
+// https://docs.gitlab.com/ee/api/merge_request_approvals.html#approve-merge-request
 type ApproveMergeRequestOptions struct {
 	Sha *string `url:"sha,omitempty" json:"sha,omitempty"`
 }

validate_test.go

@@ -53,7 +53,7 @@ func TestValidate(t *testing.T) {
 
 			mux.HandleFunc("/ci/lint", func(w http.ResponseWriter, r *http.Request) {
 				testMethod(t, r, "POST")
-				fmt.Fprintf(w, tc.response)
+				fmt.Fprint(w, tc.response)
 			})
 
 			got, _, err := client.Validate.Lint(tc.content)