1 # Copyright (c) 2009, Google Inc. All rights reserved.
3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are
7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following disclaimer
11 # in the documentation and/or other materials provided with the
13 # * Neither the name of Google Inc. nor the names of its
14 # contributors may be used to endorse or promote products derived from
15 # this software without specific prior written permission.
17 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
18 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
19 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
20 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
21 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
22 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
23 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
24 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
25 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
26 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
27 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
29 # WebKit's Python module for committer and reviewer validation
34 def __init__(self, name, email_or_emails, irc_nickname=None):
36 if isinstance(email_or_emails, str):
37 self.emails = [email_or_emails]
39 self.emails = email_or_emails
40 self.irc_nickname = irc_nickname
41 self.can_review = False
43 def bugzilla_email(self):
44 # FIXME: We're assuming the first email is a valid bugzilla email,
45 # which might not be right.
49 return '"%s" <%s>' % (self.full_name, self.emails[0])
52 class Reviewer(Committer):
54 def __init__(self, name, email_or_emails, irc_nickname=None):
55 Committer.__init__(self, name, email_or_emails, irc_nickname)
56 self.can_review = True
59 # This is intended as a canonical, machine-readable list of all non-reviewer
60 # committers for WebKit. If your name is missing here and you are a committer,
61 # please add it. No review needed. All reviewers are committers, so this list
62 # is only of committers who are not reviewers.
65 committers_unable_to_review = [
66 Committer("Aaron Boodman", "aa@chromium.org", "aboodman"),
67 Committer("Abhishek Arya", "inferno@chromium.org", "inferno-sec"),
68 Committer("Adam Langley", "agl@chromium.org", "agl"),
69 Committer("Adrienne Walker", ["enne@google.com", "enne@chromium.org"], "enne"),
70 Committer("Albert J. Wong", "ajwong@chromium.org"),
71 Committer("Alejandro G. Castro", ["alex@igalia.com", "alex@webkit.org"]),
72 Committer("Alexander Kellett", ["lypanov@mac.com", "a-lists001@lypanov.net", "lypanov@kde.org"], "lypanov"),
73 Committer("Alexander Pavlov", "apavlov@chromium.org", "apavlov"),
74 Committer("Andre Boule", "aboule@apple.com"),
75 Committer("Andrei Popescu", "andreip@google.com", "andreip"),
76 Committer("Andrew Wellington", ["andrew@webkit.org", "proton@wiretapped.net"], "proton"),
77 Committer("Andrew Scherkus", "scherkus@chromium.org", "scherkus"),
78 Committer("Andrey Kosyakov", "caseq@chromium.org", "caseq"),
79 Committer("Andras Becsi", ["abecsi@webkit.org", "abecsi@inf.u-szeged.hu"], "bbandix"),
80 Committer("Andy Estes", "aestes@apple.com", "estes"),
81 Committer("Anthony Ricaud", "rik@webkit.org", "rik"),
82 Committer("Anton Muhin", "antonm@chromium.org", "antonm"),
83 Committer("Balazs Kelemen", "kbalazs@webkit.org", "kbalazs"),
84 Committer("Ben Murdoch", "benm@google.com", "benm"),
85 Committer("Benjamin C Meyer", ["ben@meyerhome.net", "ben@webkit.org"], "icefox"),
86 Committer("Benjamin Otte", ["otte@gnome.org", "otte@webkit.org"], "otte"),
87 Committer("Benjamin Poulain", ["benjamin.poulain@nokia.com", "ikipou@gmail.com"]),
88 Committer("Brent Fulgham", "bfulgham@webkit.org", "bfulgham"),
89 Committer("Brett Wilson", "brettw@chromium.org", "brettx"),
90 Committer("Brian Weinstein", "bweinstein@apple.com", "bweinstein"),
91 Committer("Cameron McCormack", "cam@webkit.org", "heycam"),
92 Committer("Carlos Garcia Campos", ["cgarcia@igalia.com", "carlosgc@gnome.org", "carlosgc@webkit.org"], "KaL"),
93 Committer("Carol Szabo", "carol.szabo@nokia.com"),
94 Committer("Chang Shu", "Chang.Shu@nokia.com"),
95 Committer("Chris Evans", "cevans@google.com"),
96 Committer("Chris Petersen", "cpetersen@apple.com", "cpetersen"),
97 Committer("Chris Rogers", "crogers@google.com", "crogers"),
98 Committer("Christian Dywan", ["christian@twotoasts.de", "christian@webkit.org"]),
99 Committer("Collin Jackson", "collinj@webkit.org"),
100 Committer("David Smith", ["catfish.man@gmail.com", "dsmith@webkit.org"], "catfishman"),
101 Committer("Dean Jackson", "dino@apple.com", "dino"),
102 Committer("Diego Gonzalez", ["diegohcg@webkit.org", "diego.gonzalez@openbossa.org"], "diegohcg"),
103 Committer("Dirk Pranke", "dpranke@chromium.org"),
104 Committer("Drew Wilson", "atwilson@chromium.org", "atwilson"),
105 Committer("Eli Fidler", "eli@staikos.net", "QBin"),
106 Committer("Enrica Casucci", "enrica@apple.com"),
107 Committer("Erik Arvidsson", "arv@chromium.org", "arv"),
108 Committer("Eric Roman", "eroman@chromium.org", "eroman"),
109 Committer("Evan Martin", "evan@chromium.org", "evmar"),
110 Committer("Evan Stade", "estade@chromium.org", "estade"),
111 Committer("Fady Samuel", "fsamuel@chromium.org", "fsamuel"),
112 Committer("Feng Qian", "feng@chromium.org"),
113 Committer("Fumitoshi Ukai", "ukai@chromium.org", "ukai"),
114 Committer("Gabor Loki", "loki@webkit.org", "loki04"),
115 Committer("Girish Ramakrishnan", ["girish@forwardbias.in", "ramakrishnan.girish@gmail.com"]),
116 Committer("Graham Dennis", ["Graham.Dennis@gmail.com", "gdennis@webkit.org"]),
117 Committer("Greg Bolsinga", "bolsinga@apple.com"),
118 Committer("Gyuyoung Kim", ["gyuyoung.kim@samsung.com", "gyuyoung@gmail.com", "gyuyoung@webkit.org"], "gyuyoung"),
119 Committer("Hans Wennborg", "hans@chromium.org", "hwennborg"),
120 Committer("Hayato Ito", "hayato@chromium.org", "hayato"),
121 Committer("Hin-Chung Lam", ["hclam@google.com", "hclam@chromium.org"]),
122 Committer("Ilya Tikhonovsky", "loislo@chromium.org", "loislo"),
123 Committer("Jakob Petsovits", ["jpetsovits@rim.com", "jpetso@gmx.at"], "jpetso"),
124 Committer("Jakub Wieczorek", "jwieczorek@webkit.org", "fawek"),
125 Committer("James Hawkins", ["jhawkins@chromium.org", "jhawkins@google.com"], "jhawkins"),
126 Committer("James Simonsen", "simonjam@chromium.org", "simonjam"),
127 Committer("Jay Civelli", "jcivelli@chromium.org", "jcivelli"),
128 Committer("Jens Alfke", ["snej@chromium.org", "jens@apple.com"]),
129 Committer("Jer Noble", "jer.noble@apple.com", "jernoble"),
130 Committer("Jeremy Moskovich", ["playmobil@google.com", "jeremy@chromium.org"], "jeremymos"),
131 Committer("Jessie Berlin", ["jberlin@webkit.org", "jberlin@apple.com"]),
132 Committer("Jesus Sanchez-Palencia", ["jesus@webkit.org", "jesus.palencia@openbossa.org"], "jeez_"),
133 Committer("Jocelyn Turcotte", "jocelyn.turcotte@nokia.com", "jturcotte"),
134 Committer("Jochen Eisinger", "jochen@chromium.org", "jochen__"),
135 Committer("John Abd-El-Malek", "jam@chromium.org", "jam"),
136 Committer("John Gregg", ["johnnyg@google.com", "johnnyg@chromium.org"], "johnnyg"),
137 Committer("Johnny Ding", ["jnd@chromium.org", "johnnyding.webkit@gmail.com"], "johnnyding"),
138 Committer("Joost de Valk", ["joost@webkit.org", "webkit-dev@joostdevalk.nl"], "Altha"),
139 Committer("Julie Parent", ["jparent@google.com", "jparent@chromium.org"], "jparent"),
140 Committer("Julien Chaffraix", ["jchaffraix@webkit.org", "julien.chaffraix@gmail.com"]),
141 Committer("Jungshik Shin", "jshin@chromium.org"),
142 Committer("Justin Schuh", "jschuh@chromium.org", "jschuh"),
143 Committer("Keishi Hattori", "keishi@webkit.org", "keishi"),
144 Committer("Kelly Norton", "knorton@google.com"),
145 Committer("Kent Hansen", "kent.hansen@nokia.com", "khansen"),
146 Committer("Kimmo Kinnunen", ["kimmo.t.kinnunen@nokia.com", "kimmok@iki.fi", "ktkinnun@webkit.org"], "kimmok"),
147 Committer("Kinuko Yasuda", "kinuko@chromium.org", "kinuko"),
148 Committer("Krzysztof Kowalczyk", "kkowalczyk@gmail.com"),
149 Committer("Kwang Yul Seo", ["kwangyul.seo@gmail.com", "skyul@company100.net", "kseo@webkit.org"], "kwangseo"),
150 Committer("Leandro Pereira", ["leandro@profusion.mobi", "leandro@webkit.org"], "acidx"),
151 Committer("Levi Weintraub", "lweintraub@apple.com"),
152 Committer("Lucas De Marchi", ["lucas.demarchi@profusion.mobi", "demarchi@webkit.org"], "demarchi"),
153 Committer("Luiz Agostini", ["luiz@webkit.org", "luiz.agostini@openbossa.org"], "lca"),
154 Committer("Mads Ager", "ager@chromium.org"),
155 Committer("Marcus Voltis Bulach", "bulach@chromium.org"),
156 Committer("Mario Sanchez Prada", ["msanchez@igalia.com", "mario@webkit.org"], "msanchez"),
157 Committer("Matt Delaney", "mdelaney@apple.com"),
158 Committer("Matt Lilek", ["webkit@mattlilek.com", "pewtermoose@webkit.org"]),
159 Committer("Matt Perry", "mpcomplete@chromium.org"),
160 Committer("Maxime Britto", ["maxime.britto@gmail.com", "britto@apple.com"]),
161 Committer("Maxime Simon", ["simon.maxime@gmail.com", "maxime.simon@webkit.org"], "maxime.simon"),
162 Committer("Michael Nordman", "michaeln@google.com", "michaeln"),
163 Committer("Michael Saboff", "msaboff@apple.com"),
164 Committer("Michelangelo De Simone", "michelangelo@webkit.org", "michelangelo"),
165 Committer("Mike Belshe", ["mbelshe@chromium.org", "mike@belshe.com"]),
166 Committer("Mike Fenton", ["mifenton@rim.com", "mike.fenton@torchmobile.com"], "mfenton"),
167 Committer("Mike Thole", ["mthole@mikethole.com", "mthole@apple.com"]),
168 Committer("Mikhail Naganov", "mnaganov@chromium.org"),
169 Committer("MORITA Hajime", "morrita@google.com", "morrita"),
170 Committer("Nico Weber", ["thakis@chromium.org", "thakis@google.com"], "thakis"),
171 Committer("Noam Rosenthal", "noam.rosenthal@nokia.com", "noamr"),
172 Committer("Pam Greene", "pam@chromium.org", "pamg"),
173 Committer("Patrick Gansterer", ["paroga@paroga.com", "paroga@webkit.org"], "paroga"),
174 Committer("Pavel Podivilov", "podivilov@chromium.org", "podivilov"),
175 Committer("Peter Kasting", ["pkasting@google.com", "pkasting@chromium.org"], "pkasting"),
176 Committer("Peter Varga", ["pvarga@webkit.org", "pvarga@inf.u-szeged.hu"], "stampho"),
177 Committer("Philippe Normand", ["pnormand@igalia.com", "philn@webkit.org"], "philn-tp"),
178 Committer("Pierre d'Herbemont", ["pdherbemont@free.fr", "pdherbemont@apple.com"], "pdherbemont"),
179 Committer("Pierre-Olivier Latour", "pol@apple.com", "pol"),
180 Committer("Pratik Solanki", "psolanki@apple.com", "psolanki"),
181 Committer("Renata Hodovan", "reni@webkit.org", "reni"),
182 Committer("Robert Hogan", ["robert@webkit.org", "robert@roberthogan.net", "lists@roberthogan.net"], "mwenge"),
183 Committer("Roland Steiner", "rolandsteiner@chromium.org"),
184 Committer("Satish Sampath", "satish@chromium.org"),
185 Committer("Scott Violet", "sky@chromium.org", "sky"),
186 Committer("Sergio Villar Senin", ["svillar@igalia.com", "sergio@webkit.org"], "svillar"),
187 Committer("Stephen White", "senorblanco@chromium.org", "senorblanco"),
188 Committer("Tony Gentilcore", "tonyg@chromium.org", "tonyg-cr"),
189 Committer("Trey Matteson", "trey@usa.net", "trey"),
190 Committer("Tristan O'Tierney", ["tristan@otierney.net", "tristan@apple.com"]),
191 Committer("Vangelis Kokkevis", "vangelis@chromium.org", "vangelis"),
192 Committer("Victor Wang", "victorw@chromium.org", "victorw"),
193 Committer("Vitaly Repeshko", "vitalyr@chromium.org"),
194 Committer("William Siegrist", "wsiegrist@apple.com", "wms"),
195 Committer("Xiaomei Ji", "xji@chromium.org", "xji"),
196 Committer("Yael Aharon", "yael.aharon@nokia.com"),
197 Committer("Yaar Schnitman", ["yaar@chromium.org", "yaar@google.com"]),
198 Committer("Yong Li", ["yong.li.webkit@gmail.com", "yong.li@torchmobile.com"], "yong"),
199 Committer("Yongjun Zhang", "yongjun.zhang@nokia.com"),
200 Committer("Yuta Kitamura", "yutak@chromium.org", "yutak"),
201 Committer("Yuzo Fujishima", "yuzo@google.com", "yuzo"),
202 Committer("Zhenyao Mo", "zmo@google.com", "zhenyao"),
203 Committer("Zoltan Herczeg", "zherczeg@webkit.org", "zherczeg"),
204 Committer("Zoltan Horvath", ["zoltan@webkit.org", "hzoltan@inf.u-szeged.hu", "horvath.zoltan.6@stud.u-szeged.hu"], "zoltan"),
208 # This is intended as a canonical, machine-readable list of all reviewers for
209 # WebKit. If your name is missing here and you are a reviewer, please add it.
214 Reviewer("Ada Chan", "adachan@apple.com", "chanada"),
215 Reviewer("Adam Barth", "abarth@webkit.org", "abarth"),
216 Reviewer("Adam Roben", "aroben@apple.com", "aroben"),
217 Reviewer("Adam Treat", ["treat@kde.org", "treat@webkit.org", "atreat@rim.com"], "manyoso"),
218 Reviewer("Adele Peterson", "adele@apple.com", "adele"),
219 Reviewer("Alexey Proskuryakov", ["ap@webkit.org", "ap@apple.com"], "ap"),
220 Reviewer("Alice Liu", "alice.liu@apple.com", "aliu"),
221 Reviewer("Alp Toker", ["alp@nuanti.com", "alp@atoker.com", "alp@webkit.org"], "alp"),
222 Reviewer("Anders Carlsson", ["andersca@apple.com", "acarlsson@apple.com"], "andersca"),
223 Reviewer("Andreas Kling", ["kling@webkit.org", "andreas.kling@nokia.com"], "kling"),
224 Reviewer("Antonio Gomes", ["tonikitoo@webkit.org", "agomes@rim.com"], "tonikitoo"),
225 Reviewer("Antti Koivisto", ["koivisto@iki.fi", "antti@apple.com", "antti.j.koivisto@nokia.com"], "anttik"),
226 Reviewer("Ariya Hidayat", ["ariya.hidayat@gmail.com", "ariya@sencha.com", "ariya@webkit.org"], "ariya"),
227 Reviewer("Beth Dakin", "bdakin@apple.com", "dethbakin"),
228 Reviewer("Brady Eidson", "beidson@apple.com", "bradee-oh"),
229 Reviewer("Cameron Zwarich", ["zwarich@apple.com", "cwzwarich@apple.com", "cwzwarich@webkit.org"]),
230 Reviewer("Chris Blumenberg", "cblu@apple.com", "cblu"),
231 Reviewer("Chris Marrin", "cmarrin@apple.com", "cmarrin"),
232 Reviewer("Chris Fleizach", "cfleizach@apple.com", "cfleizach"),
233 Reviewer("Chris Jerdonek", "cjerdonek@webkit.org", "cjerdonek"),
234 Reviewer(u"Csaba Osztrogon\u00e1c", "ossy@webkit.org", "ossy"),
235 Reviewer("Dan Bernstein", ["mitz@webkit.org", "mitz@apple.com"], "mitzpettel"),
236 Reviewer("Daniel Bates", "dbates@webkit.org", "dydz"),
237 Reviewer("Darin Adler", "darin@apple.com", "darin"),
238 Reviewer("Darin Fisher", ["fishd@chromium.org", "darin@chromium.org"], "fishd"),
239 Reviewer("David Harrison", "harrison@apple.com", "harrison"),
240 Reviewer("David Hyatt", "hyatt@apple.com", "hyatt"),
241 Reviewer("David Kilzer", ["ddkilzer@webkit.org", "ddkilzer@apple.com"], "ddkilzer"),
242 Reviewer("David Levin", "levin@chromium.org", "dave_levin"),
243 Reviewer("Dimitri Glazkov", "dglazkov@chromium.org", "dglazkov"),
244 Reviewer("Dirk Schulze", "krit@webkit.org", "krit"),
245 Reviewer("Dmitry Titov", "dimich@chromium.org", "dimich"),
246 Reviewer("Don Melton", "gramps@apple.com", "gramps"),
247 Reviewer("Dumitru Daniliuc", "dumi@chromium.org", "dumi"),
248 Reviewer("Eric Carlson", "eric.carlson@apple.com"),
249 Reviewer("Eric Seidel", "eric@webkit.org", "eseidel"),
250 Reviewer("Gavin Barraclough", "barraclough@apple.com", "gbarra"),
251 Reviewer("Geoffrey Garen", "ggaren@apple.com", "ggaren"),
252 Reviewer("George Staikos", ["staikos@kde.org", "staikos@webkit.org"]),
253 Reviewer("Gustavo Noronha Silva", ["gns@gnome.org", "kov@webkit.org", "gustavo.noronha@collabora.co.uk"], "kov"),
254 Reviewer("Holger Freyther", ["zecke@selfish.org", "zecke@webkit.org"], "zecke"),
255 Reviewer("James Robinson", ["jamesr@chromium.org", "jamesr@google.com"], "jamesr"),
256 Reviewer("Jan Alonzo", ["jmalonzo@gmail.com", "jmalonzo@webkit.org"], "janm"),
257 Reviewer("Jeremy Orlow", "jorlow@chromium.org", "jorlow"),
258 Reviewer("Jian Li", "jianli@chromium.org", "jianli"),
259 Reviewer("John Sullivan", "sullivan@apple.com", "sullivan"),
260 Reviewer("Jon Honeycutt", "jhoneycutt@apple.com", "jhoneycutt"),
261 Reviewer("Joseph Pecoraro", ["joepeck@webkit.org", "pecoraro@apple.com"], "JoePeck"),
262 Reviewer("Justin Garcia", "justin.garcia@apple.com", "justing"),
263 Reviewer("Ken Kocienda", "kocienda@apple.com"),
264 Reviewer("Kenneth Rohde Christiansen", ["kenneth@webkit.org", "kenneth.christiansen@openbossa.org", "kenneth.christiansen@gmail.com"], "kenne"),
265 Reviewer("Kenneth Russell", "kbr@google.com", "kbr_google"),
266 Reviewer("Kent Tamura", "tkent@chromium.org", "tkent"),
267 Reviewer("Kevin Decker", "kdecker@apple.com", "superkevin"),
268 Reviewer("Kevin McCullough", "kmccullough@apple.com", "maculloch"),
269 Reviewer("Kevin Ollivier", ["kevino@theolliviers.com", "kevino@webkit.org"], "kollivier"),
270 Reviewer("Lars Knoll", ["lars@trolltech.com", "lars@kde.org", "lars.knoll@nokia.com"], "lars"),
271 Reviewer("Laszlo Gombos", "laszlo.1.gombos@nokia.com", "lgombos"),
272 Reviewer("Maciej Stachowiak", "mjs@apple.com", "othermaciej"),
273 Reviewer("Mark Rowe", "mrowe@apple.com", "bdash"),
274 Reviewer("Martin Robinson", ["mrobinson@webkit.org", "mrobinson@igalia.com", "martin.james.robinson@gmail.com"], "mrobinson"),
275 Reviewer("Mihai Parparita", "mihaip@chromium.org", "mihaip"),
276 Reviewer("Nate Chapin", "japhet@chromium.org", "japhet"),
277 Reviewer("Nikolas Zimmermann", ["zimmermann@kde.org", "zimmermann@physik.rwth-aachen.de", "zimmermann@webkit.org"], "wildfox"),
278 Reviewer("Ojan Vafai", "ojan@chromium.org", "ojan"),
279 Reviewer("Oliver Hunt", "oliver@apple.com", "olliej"),
280 Reviewer("Pavel Feldman", "pfeldman@chromium.org", "pfeldman"),
281 Reviewer("Richard Williamson", "rjw@apple.com", "rjw"),
282 Reviewer("Rob Buis", ["rwlbuis@gmail.com", "rwlbuis@webkit.org"], "rwlbuis"),
283 Reviewer("Ryosuke Niwa", "rniwa@webkit.org", "rniwa"),
284 Reviewer("Sam Weinig", ["sam@webkit.org", "weinig@apple.com"], "weinig"),
285 Reviewer("Shinichiro Hamaji", "hamaji@chromium.org", "hamaji"),
286 Reviewer("Simon Fraser", "simon.fraser@apple.com", "smfr"),
287 Reviewer("Simon Hausmann", ["hausmann@webkit.org", "hausmann@kde.org", "simon.hausmann@nokia.com"], "tronical"),
288 Reviewer("Stephanie Lewis", "slewis@apple.com", "sundiamonde"),
289 Reviewer("Steve Block", "steveblock@google.com", "steveblock"),
290 Reviewer("Steve Falkenburg", "sfalken@apple.com", "sfalken"),
291 Reviewer("Tim Omernick", "timo@apple.com"),
292 Reviewer("Timothy Hatcher", ["timothy@apple.com", "timothy@hatcher.name"], "xenon"),
293 Reviewer("Tony Chang", "tony@chromium.org", "tony^work"),
294 Reviewer(u"Tor Arne Vestb\u00f8", ["vestbo@webkit.org", "tor.arne.vestbo@nokia.com"], "torarne"),
295 Reviewer("Vicki Murley", "vicki@apple.com"),
296 Reviewer("Xan Lopez", ["xan.lopez@gmail.com", "xan@gnome.org", "xan@webkit.org"], "xan"),
297 Reviewer("Yury Semikhatsky", "yurys@chromium.org", "yurys"),
298 Reviewer("Zack Rusin", "zack@kde.org", "zackr"),
304 # Committers and reviewers are passed in to allow easy testing
307 committers=committers_unable_to_review,
308 reviewers=reviewers_list):
309 self._committers = committers + reviewers
310 self._reviewers = reviewers
311 self._committers_by_email = {}
313 def committers(self):
314 return self._committers
317 return self._reviewers
319 def _email_to_committer_map(self):
320 if not len(self._committers_by_email):
321 for committer in self._committers:
322 for email in committer.emails:
323 self._committers_by_email[email] = committer
324 return self._committers_by_email
326 def committer_by_name(self, name):
327 # This could be made into a hash lookup if callers need it to be fast.
328 for committer in self.committers():
329 if committer.full_name == name:
332 def committer_by_email(self, email):
333 return self._email_to_committer_map().get(email)
335 def reviewer_by_email(self, email):
336 committer = self.committer_by_email(email)
337 if committer and not committer.can_review: