Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

feat: set callOptions.retry properties separately#929

Merged
alexander-fenster merged 6 commits intogoogleapis:masterfrom
AVaksman:retry_options
Jan 5, 2021
Merged

feat: set callOptions.retry properties separately#929
alexander-fenster merged 6 commits intogoogleapis:masterfrom
AVaksman:retry_options

Conversation

@AVaksman
Copy link
Contributor

@AVaksman AVaksman commented Nov 4, 2020

  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes googleapis/google-cloud-node-core#392

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 4, 2020
@codecov
Copy link

codecov bot commented Nov 4, 2020

Codecov Report

Merging #929 (ef60bee) into master (a8ace3a) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     googleapis/gax-nodejs#929      +/-   ##
==========================================
- Coverage   88.52%   88.52%   -0.01%     
==========================================
  Files          43       43              
  Lines        7017     7014       -3     
  Branches      547      601      +54     
==========================================
- Hits         6212     6209       -3     
  Misses        795      795              
  Partials       10       10              
Impacted Files Coverage Δ
src/gax.ts 96.25% <100.00%> (-0.02%) ⬇️

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 a8ace3a...ef60bee. Read the comment docs.

@AVaksman AVaksman marked this pull request as ready for review November 5, 2020 14:48
@AVaksman AVaksman requested a review from a team as a code owner November 5, 2020 14:48
Copy link
Contributor

@alexander-fenster alexander-fenster left a comment

Choose a reason for hiding this comment

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

LGTM after comments are removed :)

@AVaksman AVaksman added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 19, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 19, 2020
@alexander-fenster alexander-fenster merged commit bd0e8d2 into googleapis:master Jan 5, 2021
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.

Allow users to set retryCodes and backoffSettings separately.

3 participants