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

Conversation

@igorbernstein2
Copy link
Contributor

This is extracted from https://github.com/googleapis/gax-java/pull/613/files#r229890531.
It is necessary for future integration with OpenCensus to add annotation for completed attempts

This is extracted from https://github.com/googleapis/gax-java/pull/613/files#r229890531.
It is necessary for future integration with OpenCensus to add annotation for completed attempts
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 27, 2018
@codecov-io
Copy link

Codecov Report

Merging #632 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #632      +/-   ##
===========================================
- Coverage     74.93%   74.9%   -0.04%     
  Complexity      935     935              
===========================================
  Files           177     177              
  Lines          4102    4104       +2     
  Branches        328     328              
===========================================
  Hits           3074    3074              
- Misses          875     877       +2     
  Partials        153     153
Impacted Files Coverage Δ Complexity Δ
...va/com/google/api/gax/retrying/RetryAlgorithm.java 75% <0%> (-10.72%) 7 <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 6f4ee42...2fa48f0. Read the comment docs.

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

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.

4 participants