-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Monitoring tool codecov #18457
Comments
cc @go-gitea/maintainers |
I am fine with either keeping (not bad) or removing (no help, maybe no one really looks into it) |
I am in favor of removing it entirely for the following reasons: |
The coverage is slightly different each time the test-suite is run due to the inherent non-determinism in the integration test cases. The problem is that no-one (including myself) is looking at the coverage or trying to actively increase it or work on preventing coverage getting lower. |
I personally find the report on pulls useless but tho monitor the trend important... |
Discussion:
-> #18443 (comment)
-> #18443 (review)
The text was updated successfully, but these errors were encountered: