cmd: kubectl: remove golint_failures entry

`.golint_failures` currently has an entry for `cmd/kubectl/app`. We can lint this package and remove the entry. There is only one `golint` warning; comment on exported function Run should be of the form "Run ..." Fix documentation comment and remove `cmd/kubectl/app` from `.golint_failures`.

cmd: kubectl: remove golint_failures entry
fa2de842 · tcharding · 6d1f7339 · fa2de842 · fa2de842
Commit fa2de842 authored Oct 03, 2017 by tcharding
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

kubectl.go cmd/kubectl/app/kubectl.go +2 -0

.golint_failures hack/.golint_failures +0 -1

No files found.
--- a/cmd/kubectl/app/kubectl.go
+++ b/cmd/kubectl/app/kubectl.go
@@ -31,6 +31,8 @@ import (
 WARNING: this logic is duplicated, with minor changes, in cmd/hyperkube/kubectl.go
 Any salient changes here will need to be manually reflected in that file.
 */
+// Run runs the kubectl program (creates and executes a new cobra command).
 func Run() error {
 	logs.InitLogs()
 	defer logs.FlushLogs()

--- a/hack/.golint_failures
+++ b/hack/.golint_failures
@@ -9,7 +9,6 @@ cmd/kube-proxy/app
 cmd/kubeadm/app
 cmd/kubeadm/app/apis/kubeadm
 cmd/kubeadm/app/apis/kubeadm/v1alpha1
-cmd/kubectl/app
 cmd/kubelet/app
 cmd/kubelet/app/options
 cmd/kubemark