search

Workiva / goverge

A golang multi package coverage reporting tool.
Other
3 stars 8 forks source link

Report count coverage rather then set and only run 4 threads at a time #12

Closed wesleybalvanz-wf closed 8 years ago

wesleybalvanz-wf commented 8 years ago

@Workiva/dtbackend

codecov-io commented 8 years ago

Current coverage is 88.66%

Merging #12 into master will increase coverage by +0.66% as of 0e81f4f

@@            master     #12   diff @@
======================================
  Files            7       7       
  Stmts          300     300       
  Branches        24      23     -1
  Methods          0       0       
======================================
+ Hit            264     266     +2
+ Partial          5       4     -1
+ Missed          31      30     -1

Review entire Coverage Diff as of 0e81f4f

Powered by Codecov. Updated on successful CI builds.

alexandercampbell-wk commented 8 years ago

+1 Just curious, what's the advantage of making this change?

wesleybalvanz-wf commented 8 years ago

@alexandercampbell-wf With this change we can not only see that a line is covered but also how many times it is covered, the hope is we can start identifying areas that we have a lot of test duplication and remove some duplicates to speed up our test runs.

wesleybalvanz-wf commented 8 years ago

@alexandercampbell-wf I made a small change to only run verbosely when generating XML because running verbosely causes some issues with go files that are expecting flags.

rosshendrickson-wf commented 8 years ago

+1