cytomining / pycytominer

Python package for processing image-based profiling data
https://pycytominer.readthedocs.io
BSD 3-Clause "New" or "Revised" License
80 stars 36 forks source link

ci(codecov): fix errors with automated coverage #432

Closed d33bs closed 2 months ago

d33bs commented 2 months ago

Description

This PR addresses an issue we were having with Codecov running code coverage reports from GitHub Actions. Errors were silently failed due to how they were configured. The errors which appear to have been regarding rate limiting were fixed by updating the version of the Codecov Action which is used to the latest major revision. Successful runs can be exhibited within this PR (see the commenter below).

Moving forward I've "unsilenced" this step within CI to better observe errors in the future.

Closes #431

What is the nature of your change?

Checklist

Please ensure that all boxes are checked before indicating that a pull request is ready for review.

codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.72%. Comparing base (c782728) to head (7149c08). Report is 29 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #432 +/- ## ========================================== - Coverage 94.72% 94.72% -0.01% ========================================== Files 56 57 +1 Lines 3147 3145 -2 ========================================== - Hits 2981 2979 -2 Misses 166 166 ``` | [Flag](https://app.codecov.io/gh/cytomining/pycytominer/pull/432/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cytomining) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/cytomining/pycytominer/pull/432/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cytomining) | `94.72% <ø> (-0.01%)` | :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=cytomining#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.

d33bs commented 2 months ago

Thanks @gwaybio ! Merging this in now.