search

maproulette / maproulette3

MapRoulette, the micro-tasking tool for OpenStreetMap
https://maproulette.org
MIT License
128 stars 35 forks source link

Prevent codecov from polluting PRs with useless comments #2445

Closed jake-low closed 1 month ago

jake-low commented 2 months ago

I find it really annoying that Codecov posts a comment on every single PR review (triggering a notification). And I find it extremely hard to review other people's PRs in the web UI when Codecov has put a big warning next to every third line of code saying that it's not covered (like in the screenshot below).

So I'm proposing to silence its comments on PRs. You can still view the coverage report in the "Checks" tab of the PR.

Discussion welcome if you feel that these features are useful and don't want them turned off.

image
CollinBeczak commented 1 month ago

agreed, until there are a significant amount of unit tests, and there is an effect to add them, this messages do not help review.