Welcome the new Toolkit peers

创业投资 2015-01-17

I have been a little lax in my duty of keeping the list of peers for Toolkit up to date and there have been a few notable exceptions. Thankfully we’re good about disregarding rules when it makes sense to and so many people who should be peers have already been doing reviews. Of course that’s no use to new contributors trying to figure out who should review what so I am grateful to someone who prodded me into updating the list.

As I was doing so I came to the conclusion that there is a lot of overlap between Firefox code and Toolkit code. Lots of patches touch both at the same time and it often doesn’t make a lot of sense to require different reviewers there. I also couldn’t think of a reason why someone would be a trusted reviewer of Firefox code and yet not be trusted to review Toolkit code. Looking at the differences in the lists of peers confirmed that all those missing really should be Toolkit peers too.

So going forwards I have decided that Firefox peers will automatically be considered to be Toolkit peers. That means I can announce a whole bunch of new people who are now Toolkit peers, please congratulate them in the usual way, by flooding their review queue:

  • Ehsan Akhgari
  • Mike de Boer
  • Mike Conley
  • Georg Fritzsche
  • Mark Hammond
  • Felipe Gomes
  • Gijs Kruitbosch
  • Florian Quèze
  • Tim Taubert

You might ask if the reverse should hold true, should all Toolkit peers be Firefox peers? i.e. should we just merge the lists. I leave that to the Firefox owner to decide but I will say that there are a few pieces of Toolkit that are very much not front-end and so in some cases I could see a reviewer for that area not needing to be listed in the Firefox list, not because they wouldn’t be trusted to turn down the patches they couldn’t review but just because there would be almost no patches in their area in Firefox. Maybe that case is so rare that it isn’t worth the hassle of two lists though.

Oxymoronical

责编内容by:Oxymoronical (源链)。感谢您的支持!

您可能感兴趣的

Firefox 27 Bug Statistics I’m writing today to present the bug statistics for Firefox 27. My apologies for...
Firefox Focus can lock your tabs behind Face ID If you use Firefox Focus for itsprivacy features, you'd love its latest ver...
Firefox for iOS update introduces new productivity... Today, Mozilla announced the latest version of Firefox for iOS - v...
Mozilla Delivers “the Best Firefox Ever” This week, Mozilla announced Firefox 54, which provides multi-processing cap...
Firefox Performance Update #8 Howdy folks! Another Firefox Performance Update coming at you. Buckle up. B...