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

Update micrometer dependency and fix test #650

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

objectiser
Copy link
Contributor

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

Which problem is this PR solving?

Updating the micrometer dependency version.

Short description of the changes

Changed version in build.gradle and fixed a test that fails with the newer version

@codecov
Copy link

codecov bot commented Aug 29, 2019

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #650      +/-   ##
============================================
+ Coverage     89.38%   89.62%   +0.24%     
- Complexity      563      565       +2     
============================================
  Files            69       69              
  Lines          2073     2073              
  Branches        263      263              
============================================
+ Hits           1853     1858       +5     
+ Misses          138      134       -4     
+ Partials         82       81       -1
Impacted Files Coverage Δ Complexity Δ
...gertracing/internal/reporters/LoggingReporter.java 90.9% <0%> (+9.09%) 5% <0%> (+1%) ⬆️
...rtracing/internal/reporters/CompositeReporter.java 100% <0%> (+28.57%) 7% <0%> (+1%) ⬆️

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 f1a7ca9...4ab8fcc. Read the comment docs.

Signed-off-by: Gary Brown <gary@brownuk.com>
@objectiser objectiser merged commit 21ea7af into jaegertracing:master Aug 29, 2019
@objectiser objectiser deleted the micrometer branch August 29, 2019 11:48
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