icesat2py / icepyx

Python tools for obtaining and working with ICESat-2 data
https://icepyx.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
201 stars 100 forks source link

update codecov uploader to start getting coverage reports #496

Closed JessicaS11 closed 5 months ago

JessicaS11 commented 5 months ago

At some point we stopped getting code testing coverage reports (it was failing silently in Travis CI). This PR updates to use the most recent uploader. Addresses #485.

References:

github-actions[bot] commented 5 months ago

Binder :point_left: Launch a binder notebook on this branch for commit 9b45e9f8528c5d7dedfa9f8c7b8b883655a239e8

I will automatically update this comment whenever this PR is modified

codecov[bot] commented 5 months ago

Codecov Report

Attention: 254 lines in your changes are missing coverage. Please review.

Comparison is base (d9a9d94) 59.09% compared to head (9b45e9f) 66.09%. Report is 44 commits behind head on development.

Files Patch % Lines
icepyx/core/read.py 26.59% 68 Missing and 1 partial :warning:
icepyx/core/is2ref.py 32.72% 37 Missing :warning:
icepyx/core/variables.py 26.82% 30 Missing :warning:
icepyx/quest/dataset_scripts/argo.py 79.59% 22 Missing and 8 partials :warning:
icepyx/quest/quest.py 49.09% 28 Missing :warning:
icepyx/core/query.py 69.23% 13 Missing and 7 partials :warning:
icepyx/core/auth.py 76.74% 7 Missing and 3 partials :warning:
icepyx/core/temporal.py 91.30% 4 Missing and 4 partials :warning:
icepyx/core/granules.py 22.22% 6 Missing and 1 partial :warning:
icepyx/tests/test_behind_NSIDC_API_login.py 0.00% 7 Missing :warning:
... and 4 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## development #496 +/- ## =============================================== + Coverage 59.09% 66.09% +7.00% =============================================== Files 30 36 +6 Lines 2264 3115 +851 Branches 488 552 +64 =============================================== + Hits 1338 2059 +721 - Misses 843 963 +120 - Partials 83 93 +10 ```

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