-
-
Notifications
You must be signed in to change notification settings - Fork 398
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ProjectConf: dogfooding #54
Conversation
0c683b3
to
f85cb56
Compare
f85cb56
to
8551eb2
Compare
Current coverage is 79.71% (diff: 100%)@@ master #54 diff @@
==========================================
Files 10 10
Lines 710 710
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 566 566
Misses 100 100
Partials 44 44
|
@@ -10,6 +10,8 @@ import ( | |||
"testing" | |||
) | |||
|
|||
var _ = fmt.Sprintf("", 1, 2, 3) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[govet] reported by reviewdog 🐶
no formatting directive in Sprintf call
@@ -120,6 +120,10 @@ func (w *Reviewdog) Run(r io.Reader) error { | |||
return nil | |||
} | |||
|
|||
var unused = "" | |||
|
|||
var NoComment = "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[golint] reported by reviewdog 🐶
exported var NoComment should have comment or be unexported
@@ -120,6 +120,10 @@ func (w *Reviewdog) Run(r io.Reader) error { | |||
return nil | |||
} | |||
|
|||
var unused = "" | |||
|
|||
var NoComment = "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[golint] reported by reviewdog 🐶
exported var NoComment should have comment or be unexported
This reverts commit c72964f.
ref #47