Add distinction for review made by himself.
I heard some people like to review their changes themselves. :)
[email protected]
BUG=
TEST=
Review URL: http://codereview.chromium.org/8045007
git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@102670 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/my_reviews.py b/my_reviews.py
index 1d23772..47db1da 100755
--- a/my_reviews.py
+++ b/my_reviews.py
@@ -67,12 +67,13 @@
self.multiple_lgtms = 0
self.drive_by = 0
self.not_requested = 0
+ self.self_review = 0
self.percent_done = 0.
self.percent_lgtm = 0.
self.percent_drive_by = 0.
self.percent_not_requested = 0.
- self.days = None
+ self.days = 0
self.review_per_day = 0.
self.review_done_per_day = 0.
@@ -168,7 +169,11 @@
"""Process an issue and prints stats about it."""
stats.total += 1
_process_issue(issue)
- if any(msg['sender'] == reviewer for msg in issue['messages']):
+ if issue['owner_email'] == reviewer:
+ stats.self_review += 1
+ latency = '<self review>'
+ reviewed = ''
+ elif any(msg['sender'] == reviewer for msg in issue['messages']):
reviewed = _process_issue_lgtms(issue, reviewer, stats)
latency = _process_issue_latency(issue, reviewer, stats)
else:
@@ -215,8 +220,9 @@
stats.finalize(first_day, last_day)
print >> sys.stderr, (
- '%s reviewed %d issues out of %d (%1.1f%%).' %
- (reviewer, stats.actually_reviewed, stats.total, stats.percent_done))
+ '%s reviewed %d issues out of %d (%1.1f%%). %d were self-review.' %
+ (reviewer, stats.actually_reviewed, stats.total, stats.percent_done,
+ stats.self_review))
print >> sys.stderr, (
'%4.1f review request/day during %3d days (%4.1f r/d done).' % (
stats.review_per_day, stats.days, stats.review_done_per_day))
