andymeneely / chromium-history

Scripts and data related Chromium's history
11 stars 4 forks source link

Owner can't also be a Reviewer #148

Closed andymeneely closed 10 years ago

andymeneely commented 10 years ago

When processing the reviewers in the CodeReviewParser, ignore any reviewers who are also the review owners themselves. Be sure to check the dev_id field, not the emails.

From #144, we know that this is uncommon, but it does happen over 5,000 times so it's worth making this correction.