Hi Kevin,

Thank you for sending the details.
I would like to add few more points.
There are some gerrit reviews which donot have any activity for more than 3.5 
months.
Example,https://review.tizen.org/gerrit/#/c/13170/

There are some reviews which are pending with the maintainers to merge the code 
from past one month.
For example, https://review.tizen.org/gerrit/#/c/16634/


Though the issue is Resolved from the developer point of view,the issue is 
pending with the maintainer.
For some of the modules, the maintainers are not known, it has group 
maintainers.
Some of the maintainers for few modules are changed recently. I feel that list 
has not yet been updated in the gerrit.


I suggest it would be good if an auto generated notification goes after 2 days 
if the maintainer 
delays to review/integrate the code.
I could see many of the Jira Issues can be moved to resolved state if the 
gerrit review process ends fastly.

Thanks & Regards,
Rajendra.


----------------------------------------------------------------------

Message: 1
Date: Fri, 21 Mar 2014 16:33:41 +0100
From: K?vin THIERRY  <[email protected]>
To: "[email protected]" <[email protected]>
Subject: [Dev] [Gerrit-status] To-merge:17 To-review:165; To-verify:40
        To-review-and-verify:291 Invalid:22 Total:535
Message-ID: <[email protected]>
Content-Type: text/plain; charset="iso-8859-1"; Format="flowed"

Hi everybody,

Please find, attached to this mail, a list of pending Gerrit reviews. 
Only reviews which didn't receive any activity in the last two days are 
part of this list.

Summary:

To-merge: 17
To-review: 165;
To-verify: 40
To-review-and-verify: 291
Invalid: 22

Total: 535


Have the status TO_MERGE, reviews with:
- at least one code-review +2
- at least one verified +1
- no code-review -2
- no verified -1

Have the status TO_REVIEW, reviews with:
- at least one verified +1
- no code-review +2
- no code-review -2
- no verified -1

Have the status TO_VERIFY, reviews with:
- at least one code-review +2
- no code-review -2
- no verified +1
- no verified -1

Have the status TO_REVIEW_AND_VERIFY, reviews with:
- no code-review +2
- no code-review -2
- no verified +1
- no verified -1

Have the status INVALID, reviews with:
- at least one code-review -2 OR one verified -1


If I didn't make any mistake, those status should cover all opened reviews.
If you find an error in the file, please contact me so I can correct the 
scripts that generate it.

I'll probably send an updated status of the pending reviews once a week.

I wish you all a nice week-end !
Kevin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: gerrit-status-2014-03-21.csv
Type: text/csv
Size: 70944 bytes
Desc: not available
URL: 
<http://lists.tizen.org/pipermail/dev/attachments/20140321/a06a6d6d/attachment.csv>

------------------------------

<p>&nbsp;</p><p>&nbsp;</p>
_______________________________________________
Dev mailing list
[email protected]
https://lists.tizen.org/listinfo/dev

Reply via email to