1318053 - Suggested reviewers update for Core

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Administration, task)

Description

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

The list of reviewers for components in core is sparse:

https://bugzilla.mozilla.org/page.cgi?id=review_suggestions.html#Core

This needs review and updating.

Comment 1

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

FYI: A product can have a suggested set of reviewers which can be overridden at the component level.

Comment 2

[Maire Reavy [:mreavy]]

The WebRTC reviewers are out of date.  Here's the updated list of folks who regularly review these components nowadays:

WebRTC [e-mail], [e-mail], [e-mail], [e-mail]
WebRTC: Audio/Video [e-mail],  [e-mail], [e-mail], [e-mail], [e-mail]
WebRTC: Networking [e-mail], [e-mail], [e-mail], [e-mail] 
WebRTC: Signaling [e-mail], [e-mail], [e-mail], [e-mail] 


Are you also looking to add Core components to this list (ones that are missing)?

Comment 3

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Yes, I'd like to backfill the whole list of components in Core. 

I've updated the reviewer lists for WebRTC, thank you.

Would [e-mail] also be the person to triage incoming bugs in WebRTC?

Comment 4

[Maire Reavy [:mreavy]]

(In reply to Emma Humphries ☕️ [:emceeaich] (UTC-8) +needinfo me from comment #3)
> Yes, I'd like to backfill the whole list of components in Core. 
> 
> I've updated the reviewer lists for WebRTC, thank you.
> 
> Would [e-mail] also be the person to triage incoming bugs in WebRTC?

We actually distribute bug triage among our team members on a daily basis.  I and Jesup are the co-leads for that effort (i.e. if there is a particularly tricky bug to triage, the person on triage for that day will ping one of us to ask for a priority call).

Jesup is the module owner for WebRTC; I'm the engineering manager.

Comment 5

[Maire Reavy [:mreavy]]

Note: [e-mail] - only one "s" in jesup (ref: comment 3)

Comment 6

[Milan Sreckovic [:milan] (needinfo for best results)]

We can start with this.

Canvas: 2D - [e-mail], [e-mail], [e-mail], [e-mail]

Canvas: WebGL - [e-mail], [e-mail], 

GFX: Color Management - [e-mail]

Graphics - [e-mail], [e-mail], [e-mail], [e-mail], [e-mail], [e-mail], [e-mail], [e-mail], 

Graphics: Layers - [e-mail], [e-mail], [e-mail], [e-mail], [e-mail], 

Graphics: Text - [e-mail], 

Graphics: WebRender - [e-mail], [e-mail], [e-mail], 

ImageLib - [e-mail], [e-mail], 

Panning and Zooming - [e-mail], [e-mail],

Comment 7

[Andrew Overholt [:overholt]]

Elsewhere I've asked if we could automate this somehow but here are some suggestions for DOM-y components:

Document Navigation - :bz, :smaug
DOM - :qDot, :bkelly, :baku, :peterv, :bz, :ehsan, :smaug
DOM: Core & HTML - :peterv, :bz, :ehsan, :smaug, :bkelly
DOM: CSS Object Model - :heycam, :bz
DOM: Device Interfaces - :qDot
DOM: Events - :smaug, :stone, :masayuki
DOM: IndexedDB - :janv, :bevistseng, 
DOM: Push Notifications - :kitcambridge, :wchen
DOM: Service Workers - :bkelly, :asuth, :ehsan, :nsm
DOM: Workers - :baku, :bkelly, :bent
Event Handling - :smaug, :masayuki
Geolocation - :jdm, :kanru
HTML: Form Submission - :baku, :bz, :smaug
HTML: Parser - :hsivonen, :wchen
Keyboard: Navigation - :masayuki, :smaug
Serializers - :hsivonen
XBL - :bholley, :mrbkap, :bz, :smaug
XML - :peterv, :bz, :hsivonen
XPConnect - :bholley, :mrbkap, :bz
XSLT - :peterv, :sicking

Comment 8

[Andrew Overholt [:overholt]]

+ :bent, :sicking for IndexedDB

Comment 9

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Added the above changes. I'm going to close this and let's continue in bug 1318458.