-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TST: Turn off codecov project reporting #16460
base: main
Are you sure you want to change the base?
Conversation
because it is meaningless and almost always fails now for no reason.
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
is there any value in keeping this configuration file at all if everything is turned off ? For the record: I support ditching codecov, it hasn't been reliable for a while, and has degraded so much that we almost always just ignore it. |
I think we still need it to turn things off, right? Otherwise, it pulls codecov defaults? Ditching it is an upstream issue, please see OpenAstronomy/github-actions-workflows#189 |
If I understand correctly the upstream issue is about supporting report uploads natively in github actions, but it seems already possible to disable report upload altogether without preserving diff --git a/.github/workflows/ci_workflows.yml b/.github/workflows/ci_workflows.yml
index 9980b78510..f244528364 100644
--- a/.github/workflows/ci_workflows.yml
+++ b/.github/workflows/ci_workflows.yml
@@ -64,7 +64,7 @@ jobs:
# run only with minimal dependencies.
- name: Python 3.12 with minimal dependencies and full coverage
linux: py312-test-cov
- coverage: codecov
+ coverage: ''
- name: Python 3.11 in Parallel with all optional dependencies
linux: py311-test-alldeps-fitsio
@@ -79,7 +79,7 @@ jobs:
- name: Python 3.10 with oldest supported version of all dependencies
linux: py310-test-oldestdeps-alldeps-cov-clocale
posargs: --remote-data=astropy
- coverage: codecov
+ coverage: ''
- name: Python 3.11 with all optional dependencies (Windows)
windows: py311-test-alldeps |
I also wonder if CI jobs that have the |
@neutrinoceros , no I think the OpenAstronomy issue is about ditching codecov and upload coverage.xml only with Actions. But maybe @Cadair can confirm. |
I think we still want to keep parch reporting right? |
Yes, I think patch report still useful (for now). And you can still see that in this PR. |
@neutrinoceros is correct if you don't want to upload coverage in a job you can disable it as in the example. However, I don't know if just not uploading anything to codecov is enough to shut it up for if you need to remove the app or disable it in config. |
It's funny. In many of my repos I have similar issues with Codecov, but it's the patch cov that's flaky and the project coverage that I still use. IDK what's up with codecov. |
👍 to removing
|
Codecov does have a nice UI though. |
Description
This pull request is to turn off codecov project reporting because it is meaningless and almost always fails now for no reason. This was requested by @astrofrog
Ref: https://docs.codecov.com/docs/commit-status#disabling-a-status