New issue
Advanced search Search tips

Issue 780472 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 568949
Owner: ----
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Use code coverage report data from bots in CL view

Project Member Reported by mathias@chromium.org, Nov 1 2017

Issue description

we have build bots that produce coverage reports, e.g. https://storage.googleapis.com/chromium-v8/linux64_gcov_rel/35b6aa38497fdd1ea08af7d4042528c645826b06/index.html We currently don’t have alerts set up for this. We also do not enforce a specific % of code coverage (which is fine!).

However, it would be nice if % coverage change per file would be displayed on https://chromium-review.googlesource.com/ for each CL once the bots complete their runs. This way, engineers can get an idea of the impact of their CL on code coverage before it lands, without having to manually navigate to the coverage report every single time.

Here’s a similar example of this kind of thing on GitHub: https://github.com/mathiasbynens/he/pull/55#issuecomment-302787905

 
Owner: ----
Status: Available (was: Untriaged)
Mergedinto: 568949
Status: Duplicate (was: Available)
We had that. Kind of died with rietveld, and was never really accepted/used by developers. We can discuss if a revival in gerrit has merrit.

Sign in to add a comment