bcgsc / pori_python

GNU General Public License v3.0
0 stars 1 forks source link

Feature/devsu 2349 add async report loading #4

Closed elewis2 closed 4 months ago

elewis2 commented 4 months ago

Adds option to upload reports asynchronously, setting the number of mins the outcome should be waited for. Default is five minutes (plus an initial 3 minutes of checks with shorter times between them).

Option to use async and number of minutes to wait is settable in upload_report and ipr.main.

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 16.66667% with 25 lines in your changes missing coverage. Please review.

Project coverage is 86.17%. Comparing base (cdf775f) to head (0e0aa5c).

Files Patch % Lines
pori_python/ipr/connection.py 13.79% 25 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #4 +/- ## =========================================== - Coverage 87.13% 86.17% -0.96% =========================================== Files 19 19 Lines 2129 2156 +27 =========================================== + Hits 1855 1858 +3 - Misses 274 298 +24 ``` | [Flag](https://app.codecov.io/gh/bcgsc/pori_python/pull/4/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgsc) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/bcgsc/pori_python/pull/4/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgsc) | `86.17% <16.66%> (-0.96%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=bcgsc#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.