lint: exclude vendor directory #67

Merged
merged 1 commit into from Jan 7, 2017

Projects

None yet

3 participants

@haya14busa
Owner

No description provided.

@haya14busa lint: exclude vendor directory
b2558a4
@codecov-io
codecov-io commented Jan 7, 2017 edited

Current coverage is 79.68% (diff: 100%)

Merging #67 into master will not change coverage

@@             master        #67   diff @@
==========================================
  Files            10         10          
  Lines           753        753          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            600        600          
  Misses          105        105          
  Partials         48         48          

Powered by Codecov. Last update 08050fc...b2558a4

reviewdog.go
@@ -11,6 +11,8 @@ import (
"github.com/haya14busa/reviewdog/diff"
)
+var unused = 1
@reviewdog
reviewdog Jan 7, 2017

[unused] reported by reviewdog 🐶
var unused is unused (U1000)

@haya14busa
haya14busa Jan 7, 2017 Owner

Nice 👍

@haya14busa haya14busa merged commit d6e617f into master Jan 7, 2017

7 checks passed

ci/circleci Your tests passed on CircleCI!
Details
codecov/patch Coverage not affected when comparing 08050fc...b2558a4
Details
codecov/project 79.68% (+0.00%) compared to 08050fc
Details
continuous-integration/drone the build was successful
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 82.492%
Details
@haya14busa haya14busa deleted the exclude-vendor branch Jan 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment