FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added sleep between upload retries. by nmoinvaz · Pull Request #295 · codecov/codecov-python · GitHub

This repository was archived by the owner on May 21, 2025. It is now read-only.
/ codecov-python Public archive

Added sleep between upload retries. - #295

Merged
thomasrockhu merged 3 commits into
codecov:masterfrom
nmoinvaz:improvements/retry-wait
Sep 14, 2020
Merged

Added sleep between upload retries.#295
thomasrockhu merged 3 commits into
codecov:masterfrom
nmoinvaz:improvements/retry-wait

Conversation

Copy link
Copy Markdown
Contributor

See #293.

nmoinvaz commented Aug 30, 2020
edited
Loading

Copy link
Copy Markdown
Contributor Author

Not sure why the failure on black on a single instance. I checked manually and that file tests\test.py appears fine which I don't even touch in this PR.

nmoinvaz changed the title Added sleep between upload retries with increasing wait. Added sleep between upload retries. Aug 31, 2020

Copy link
Copy Markdown
Contributor Author

This PR:

  • Adds sleep 2s for each retry (same as codecov bash uploader)
  • Sets default retries to 5 (same as codecov bash uploader)
  • Logs retry attempts to output for debugging.

Comment thread codecov/__init__.py Outdated

codecov Bot commented Sep 14, 2020

Copy link
Copy Markdown

Codecov Report

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

@@          Coverage Diff          @@
##           master   #295   +/-   ##
=====================================
  Coverage      88%    88%           
=====================================
  Files           2      2           
  Lines           9      9           
=====================================
  Hits            8      8           
  Misses          1      1           

Copy link
Copy Markdown
Contributor Author

I have fixed the CI issue.

thomasrockhu merged commit 2574d1b into codecov:master Sep 14, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL