Hi, Tom from Codecov here. I noticed that you were using Codecov but weren't actually getting any notifications on pull requests. I figured it would be useful to get some idea if code being changed is being tested, but also not blocking CI/merging. Let me know if this makes sense or if we can do something that would be helpful.
This commit disables the Codecov notification whenever a commit is added
to the PR reporting if the change passed or not Codecov checks for code
coverage. Doing this after discussion with the rest of the team as we
feel there are a significant number of false positives in Codecov
failures.
This commit disables the comment that Codecov leaves in each PR
reporting on code coverage status. We decided to do this because those
comments included a lot of information that is not useful for our
project and often included false positives reports of reduction in the
code coverage.
This commit is an attempt to reduce the noise in PRs generated by the code coverage comment posted automatically by Codecov. It changes Codecov configuration to post a comment to a PR only if it changes the code coverage and it also removes reach graph, the diff image and the flags from the comment content. Leaving only a list of the files impacted by the PR. My initial plan was to leave in the comment content only a link to the full report, but I couldn't find a way to do that in the documentation (https://docs.codecov.io/docs/pull-request-comments).
Sam Seay
Tom Hu
Rodrigo Primo
Mike Jolley