It would be nice to infer coverage for partial builds from branch master.
We run a mono repo and we don’t always run coverage for every sub repo on every PR to save CI time.
We upload reports with -F <topic> to specify a topic. It would be nice if codecov would infer coverage for the other -F <topic> as the coverage of the branch commit if nothing is reported before the timeout.
This would make the coverage accurate with respect to a merge to master.
Great idea @nickpalmer!! We are calling this “carryforward” flags where we carryforward any coverage not updated from wherever it was most recently run. Looking at building this in the next several sprint cycles.
I’m working on a project using dynamic build pipelines (via Buildkite), and we are tagging all of our coverage submissions using -F <flags>. This would be incredibly helpful to us, as currently have had to turn off coverage diff messages since they don’t contain accurate data, and it would be a very useful thing to be able to alert the team on coverage regressions!
Hey @nickpalmer@JCMais@ccarpita – this feature for partial test runs “Carryforward Flags” is now in production in beta, would any of you like to test it out?
We are not getting what I would expect with carry forward turned on in the repository I am attempting to use it on.
On a recent change that swapped 3 ‘let’ statements for ‘const’ statements across two flags in areas which are uncovered, we do see that it correctly identified the delta of zero in those three files. However, the coverage diff does not seem to be carrying forward, and shows -12.19% as an overall coverage drop.
Thanks you! That fixed it for us. Is there a linter for codecov.yml that would reveal these issues, or a place to see parse errors in the UI? I searched docs for that but didn’t find it.
Hi @nickpalmer, we are working on getting more of these issues user-facing, but as of now there isn’t. We are planning to get a better yaml validator out soon.
@tom Awesome thanks. I did find the validator we can post to, but it just seems to check if the yaml is valid and return it as json, not validate it against a schema.
I posted a new support issue around multiple comments on our PRs which I am not sure if that is related to carry forward, but it might be.