search

weaveworks / mesh

A tool for building distributed applications.
Apache License 2.0
880 stars 107 forks source link

Minor changes prompted by lint #74

Closed bboreham closed 7 years ago

bboreham commented 7 years ago

Evidently gosimple has added more checks, so make these changes to keep it happy.

marccarre commented 7 years ago

Fixes:

  1. connection_maker.go:146:36:warning: should use make(chan []string) instead (S1019) (gosimple)
  2. status.go:161:51:warning: should use make(chan []LocalConnectionStatus) instead (S1019) (gosimple)
  3. token_bucket.go:33:13:warning: should use time.Until instead of t.Sub(time.Now()) (S1024) (gosimple)
  4. the following warnings: 
    WARNING: deadline exceeded by linter errcheck on . (try increasing --deadline)
WARNING: deadline exceeded by linter staticcheck on . (try increasing --deadline)
WARNING: deadline exceeded by linter structcheck on . (try increasing --deadline)

    See also: https://circleci.com/gh/weaveworks/mesh/346