Skip to content
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

Discovery: Investigate Feasibility of Incremental Reporting for Patch Coverage #3020

Open
codecovdesign opened this issue Nov 26, 2024 · 0 comments

Comments

@codecovdesign
Copy link
Contributor

Context

In #2702 we are working on improving how and when the report is shown in the PR comment and app to provide clearer and more accurate information. From #2922, we decided v1 is the following scenarios updated / reporting paradigm:

  1. If no error and status is pending:

    • Do not display a comment until the report is finalized
    • Await all uploads and show the report only when fully processed
  2. If there is an error:

    • Display the error message in the PR comment
    • Do not show the partial report

As a next step, we want to explore the ability to incrementally report on patch coverage as uploads are successfully parsed. The concept is to:

  • Receive confirmation for specific lines in the patch as uploads are processed
  • Update the report incrementally with patch coverage information, showing which lines are covered by tests
  • Wait to display the full overall report (e.g., project coverage) until all uploads are successfully processed

This way the user receives the most important data point – patch – and can move on in absense of the remaining report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant