Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

Conversation

@andreamlin
Copy link
Contributor

For protobuf and grpc

For protobuf and grpc
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 25, 2019
@codecov
Copy link

codecov bot commented Apr 25, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #709   +/-   ##
=========================================
  Coverage     75.46%   75.46%           
  Complexity     1037     1037           
=========================================
  Files           196      196           
  Lines          4675     4675           
  Branches        363      363           
=========================================
  Hits           3528     3528           
  Misses          986      986           
  Partials        161      161

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 8a0df4a...bdeb66b. Read the comment docs.

@andreamlin andreamlin requested a review from vam-google April 25, 2019 16:28
@andreamlin
Copy link
Contributor Author

PTAL

Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM under assumption that bazel test //... works

@andreamlin andreamlin merged commit 13564bb into master Apr 29, 2019
@elharo elharo deleted the deps branch October 29, 2020 14:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants