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

Define okhttp version in top level build.gradle, along with other dep… #402

Merged
merged 1 commit into from
Apr 19, 2018

Conversation

objectiser
Copy link
Contributor

…endencies

Follow on from #397 - to include missed okhttp dependency.

Signed-off-by: Gary Brown gary@brownuk.com

…endencies

Signed-off-by: Gary Brown <gary@brownuk.com>
@ghost ghost assigned objectiser Apr 19, 2018
@ghost ghost added the review label Apr 19, 2018
@objectiser objectiser requested a review from jpkrohling April 19, 2018 10:17
@codecov
Copy link

codecov bot commented Apr 19, 2018

Codecov Report

Merging #402 into master will decrease coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master     #402     +/-   ##
===========================================
- Coverage     84.45%   84.25%   -0.2%     
  Complexity      500      500             
===========================================
  Files            66       66             
  Lines          2033     2033             
  Branches        247      247             
===========================================
- Hits           1717     1713      -4     
- Misses          235      239      +4     
  Partials         81       81
Impacted Files Coverage Δ Complexity Δ
...in/java/io/jaegertracing/senders/ThriftSender.java 75% <0%> (-4.55%) 10% <0%> (ø)
...ava/io/jaegertracing/reporters/RemoteReporter.java 84.7% <0%> (-2.36%) 8% <0%> (ø)

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 acb2a24...09177f4. Read the comment docs.

@jpkrohling jpkrohling merged commit 09177f4 into jaegertracing:master Apr 19, 2018
@ghost ghost removed the review label Apr 19, 2018
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.

2 participants