Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vendor: pin "grpc/grpc-go" v1.7.5 #9042

Merged
merged 1 commit into from
Dec 19, 2017
Merged

vendor: pin "grpc/grpc-go" v1.7.5 #9042

merged 1 commit into from
Dec 19, 2017

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Dec 18, 2017

No need backport. Just to explicitly pin the latest v1.7.x release.

/cc @jpbetz

Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
@gyuho gyuho requested review from jpbetz and xiang90 December 18, 2017 23:57
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@a7445d7). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #9042   +/-   ##
=========================================
  Coverage          ?   75.99%           
=========================================
  Files             ?      359           
  Lines             ?    29838           
  Branches          ?        0           
=========================================
  Hits              ?    22676           
  Misses            ?     5584           
  Partials          ?     1578

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 a7445d7...d57002b. Read the comment docs.

Copy link
Contributor

@jpbetz jpbetz left a comment

Choose a reason for hiding this comment

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

I agree, no need to backport

@gyuho gyuho merged commit 6f89918 into etcd-io:master Dec 19, 2017
@gyuho gyuho deleted the grpc-go branch December 19, 2017 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants