Skip to content
This repository was archived by the owner on Apr 24, 2024. It is now read-only.

adding test to retry on http error#107

Merged
lubird merged 6 commits intosegmentio:masterfrom
cgalan-applaudo:feature/LAPP-12
Feb 7, 2020
Merged

adding test to retry on http error#107
lubird merged 6 commits intosegmentio:masterfrom
cgalan-applaudo:feature/LAPP-12

Conversation

@cgalan-applaudo
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 24, 2020

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #107   +/-   ##
======================================
  Coverage    83.7%   83.7%           
======================================
  Files          32      32           
  Lines         583     583           
======================================
  Hits          488     488           
  Misses         95      95

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 a001f2b...4da2945. Read the comment docs.

@lubird lubird self-requested a review February 3, 2020 22:19
statusCode = (int)response.StatusCode;

if (response.StatusCode == HttpStatusCode.OK)
if (statusCode == (int)HttpStatusCode.OK)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Indenting issue

@lubird lubird merged commit 86b2ad8 into segmentio:master Feb 7, 2020
@cgalan-applaudo cgalan-applaudo deleted the feature/LAPP-12 branch March 6, 2020 16:54
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.

3 participants