Re: [Xen-devel] Prototype Code Review Dashboards (input required)

[Daniel Izquierdo <dizquierdo@xxxxxxxxxxxx>]

On 02/03/16 23:45, Daniel Izquierdo wrote:

[...]
> > Given, that the Xen-A1.A2.A3 dash board is quite busy, we should keep 
> > that separate.
> I still have to upload this, sorry for the delay!
>
>
> This is a summary of the actions unless extra comments are provided:
>
> * Balance should be calculated as reviews - patches
> * Cleaning actions in the dataset when finding multiple email addresses
> * Bugs with the post-ack comments
> * Add the extra panel defined as use case A.0
>
> Some extra feedback or discussion:
>
> * Reduce the flags to be used. We're currently using all of the flags 
> available and sub-setting the reviewed-by and acked-by
> * I'm not sure if we need extra discussion related to the merge:1 filter
> * Check how reviews and patches are counted. Any new version of a 
> patch is counted as a new patch. Any new reviewed-by flag in a reply 
> to a patch is counted as a review.

From those actions, this is the current status:

* Balance should be calculated as reviews - patches (DONE)
* Cleaning actions in the dataset when finding multiple email addresses 
(still in progress and I should add your comments in a previous email)
* Bugs with the post-ack comments (DONE)
* Add the extra panel defined as use case A.0 (DONE)
* Tables at the bottom of the panels contains proper information and not 
all of the fields (DONE)
* Do not count comments to your own patches (to be done)

Regarding to the extra feedback:
* Reduce the flags to be used. We're currently using all of the flags 
available and sub-setting the reviewed-by and acked-by.
** This opened a new discussion about the CC mailing lists and the 
creation of some buckets of info.
* I'm not sure if we need extra discussion related to the merge:1 filter
** No advances
* Check how reviews and patches are counted. Any new version of a patch 
is counted as a new patch. Any new reviewed-by flag in a reply to a 
patch is counted as a review.
** No advances


Regards,
Daniel.

--
Daniel Izquierdo Cortazar, PhD
Chief Data Officer
---------
"Software Analytics for your peace of mind"
www.bitergia.com
@bitergia


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel