Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Add codecov ignores #153

Merged
merged 5 commits into from
Apr 27, 2017
Merged

Add codecov ignores #153

merged 5 commits into from
Apr 27, 2017

Conversation

vprithvi
Copy link
Contributor

Also set precision to 2.

@codecov-io
Copy link

codecov-io commented Apr 27, 2017

Codecov Report

Merging #153 into master will increase coverage by 43.87%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #153       +/-   ##
=============================================
+ Coverage     36.24%   80.11%   +43.87%     
+ Complexity      598      457      -141     
=============================================
  Files            94       76       -18     
  Lines          6349     1715     -4634     
  Branches       1052      188      -864     
=============================================
- Hits           2301     1374      -927     
+ Misses         3866      255     -3611     
+ Partials        182       86       -96
Impacted Files Coverage Δ Complexity Δ
...java/com/uber/jaeger/samplers/SamplingManager.java
...main/java/com/uber/jaeger/crossdock/Constants.java
...a/com/uber/jaeger/crossdock/api/TraceResponse.java
...crossdock/resources/behavior/EndToEndBehavior.java
...ources/behavior/http/EndToEndBehaviorResource.java
...om/uber/jaeger/crossdock/api/JoinTraceRequest.java
...resources/behavior/http/TraceBehaviorResource.java
...ck/resources/behavior/tchannel/TChannelServer.java
...ore/src/main/java/com/uber/jaeger/metrics/Tag.java
...java/com/uber/jaeger/crossdock/api/Downstream.java
... and 12 more

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 9c74133...43ad36c. Read the comment docs.

codecov.yml Outdated
ignore:
- "jaeger-thrift"
- "jaeger-crossdock"
- "**/gen-java/**"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC this is only in jaeger-thrift which you are ignoring just above

- Exclude this in the gradle build so that code coverage may be checked locally as well
@vprithvi vprithvi merged commit eb5947e into master Apr 27, 2017
@vprithvi vprithvi deleted the codecov-yaml branch April 27, 2017 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants