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

upgrade GSON and slf4j-api version #663

Merged
merged 1 commit into from
Oct 18, 2019

Conversation

chandresh-pancholi
Copy link
Contributor

@chandresh-pancholi chandresh-pancholi commented Oct 17, 2019

Signed-off-by: chandresh-pancholi chandreshpancholi007@gmail.com

Resolves #651

Which problem is this PR solving?

  • upgrade GSON and slf4j-api version

Short description of the changes

Signed-off-by: chandresh-pancholi <chandreshpancholi007@gmail.com>
@codecov
Copy link

codecov bot commented Oct 17, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #663   +/-   ##
=========================================
  Coverage     89.83%   89.83%           
  Complexity      569      569           
=========================================
  Files            69       69           
  Lines          2086     2086           
  Branches        266      266           
=========================================
  Hits           1874     1874           
  Misses          131      131           
  Partials         81       81

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 fd9f743...23b6044. Read the comment docs.

@yurishkuro
Copy link
Member

@chandresh-pancholi please make sure the description not just references a ticket, but does it in the form that GitHub recognizes, e.g. Resolves #651. This way github will close the ticket once the PR is merged.

@jpkrohling jpkrohling changed the title upgrade GSON and slf4j-api version #651 upgrade GSON and slf4j-api version Oct 18, 2019
Copy link
Collaborator

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

LGTM, but I would like to get @objectiser's approval as well.

@objectiser objectiser merged commit 27d7fce into jaegertracing:master Oct 18, 2019
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.

upgrade GSON and slf4j-api version
4 participants