search

uber-go / config

Configuration for Go applications
https://godoc.org/go.uber.org/config
MIT License
448 stars 41 forks source link

Lower coverage target to 90% #96

Closed akshayjshah closed 6 years ago

akshayjshah commented 6 years ago

Our internal OSS process suggests a bar of 90% coverage. We should aim for more, but let's not block PRs until we drop below that bar.

codecov[bot] commented 6 years ago

Codecov Report

Merging #96 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #96   +/-   ##
=======================================
  Coverage   94.35%   94.35%           
=======================================
  Files           8        8           
  Lines         390      390           
=======================================
  Hits          368      368           
  Misses         14       14           
  Partials        8        8

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1691f21...063d6ed. Read the comment docs.