I should apologise, in my eagerness to get this posted I missed the rather 
significant step of going through the existing peers and making sure that they 
were aware of and had a place in the new structure. There was no intent to 
silently drop folks and I am sorry if anyone felt like that was what happened. 
I’ll be reaching out directly to the peers shortly.

> On Dec 13, 2019, at 1:03 PM, Dave Townsend <dtowns...@mozilla.com> wrote:
> 
> The Firefox and Toolkit modules together cover a large amount of code 
> encompassing the entire front-end of Firefox as well as numerous services 
> that are used by external projects, notably Thunderbird. For a while now both 
> modules have been owned by me with identical peer lists. We have also had a 
> flat structure, no specific sub-modules for different areas of the code.
> 
> This has caused a couple of challenges the main one being that from the 
> outside no-one really knows who is responsible for what and who to send 
> patches to for review, a few popular peers tend to get most of the review 
> requests. A secondary issue is that gaining peership over all of Firefox and 
> Toolkit imposed a larger barrier than necessary for most potential reviewers.
> 
> I'd like to make three changes to these modules:
> 
> 1. Rename the Firefox module to Desktop Firefox. We ship multiple products 
> called Firefox, this module is specific to the desktop version.
> 
> 2. Split the module up into a number of sub-modules with defined owners and 
> peers responsible for that area of code leaving a few top-level peers able to 
> review code that cuts across modules or doesn't fall into the list of 
> sub-modules. The list of initial sub-modules is here: 
> https://wiki.mozilla.org/User:Mossop:NewModule 
> <https://wiki.mozilla.org/User:Mossop:NewModule> (the owners are correct, the 
> list of peers needs some work). This list is not expected to be final, I 
> probably missed a few things and over time we’ll add more as needed. The 
> expectation is that the sub-module owners will be free to update the list of 
> their peers whenever they like.
> 
> 3. Since the modules cover such a large area of code I'd like to add a 
> joint-module owner to share the responsibility. There are many engineers who 
> know the code well enough but Gijs Kruitbosch has consistently been thinking 
> globally about how we implement and maintain features throughout the desktop 
> code.
> 
> You’ll note that the Toolkit module is light on sub-modules. There are many 
> features that have backend pieces in Toolkit and front-end pieces in Desktop 
> Firefox. Features like Bookmarks and History, Downloads, Password Manager. 
> These sub-modules largely exist in service to the desktop product and as such 
> are maintained by desktop engineers, so I felt it was appropriate to just 
> list them once under Desktop Firefox.
> 
> Please let me know if you have objections to this plan. Due to the size of 
> the changes I’m going to wait two weeks and then if there are no significant 
> objections I will update the module pages.

_______________________________________________
governance mailing list
governance@lists.mozilla.org
https://lists.mozilla.org/listinfo/governance

Reply via email to