SnowEx / snowexsql

A project to access the SnowEx database holding data from the NASA SnowEx campaign
https://snowexsql.readthedocs.io/en/latest/readme.html
GNU General Public License v3.0
25 stars 6 forks source link

Actions - Switch to use CodeCov to check test coverage #126

Closed jomey closed 2 months ago

jomey commented 2 months ago

Change the workflow to get test coverage and use the CodeCov service. This also separates the step into a dedicated CI action and independent of the Python 3.9 test runs. Upgrading the pytest and pytest-cov package to always use the latest.

codecov[bot] commented 2 months ago

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

micah-prime commented 2 months ago

Would there be wisdom in pinning to a major version for pytest and pytest-cov in case there are breaking changes?

jomey commented 2 months ago

Would there be wisdom in pinning to a major version for pytest and pytest-cov in case there are breaking changes?

Open to suggestions here. I am less concerned with test frameworks as we could also pin them once they really break on us. I can be convinced here to pin them regardless

micah-prime commented 2 months ago

Would there be wisdom in pinning to a major version for pytest and pytest-cov in case there are breaking changes?

Open to suggestions here. I am less concerned with test frameworks as we could also pin them once they really break on us. I can be convinced here to pin them regardless

Yeah let's just leave it and see what happens.

Do you know why the 'Actions' aren't linking under 'Checks' in the PR?

jomey commented 2 months ago

Do you know why the 'Actions' aren't linking under 'Checks' in the PR?

I done my last commit with [skip ci] since I just added the CodeCov badge. Hence it only run the docs for that one. If you look at the :heavy_check_mark: of my first commit, you will see that all actions were run on that one.