Calendar:Module Ownership: Difference between revisions
Mschroeder (talk | contribs) No edit summary |
(Add link to current calendar module page.) |
||
| (25 intermediate revisions by 7 users not shown) | |||
| Line 1: | Line 1: | ||
This is | <small>[[Calendar:Home_Page|<< Back to Calendar Home Page]]</small> | ||
This is a non-current list of owners and peers of the calendar modules. A more current list is [[Modules/Calendar|here]]. '''Don't edit this file. Just don't.''' | |||
Items without any owners at all are generally handled by those under "Default." However, in general, they could benefit from being their own strongly-owned modules at some point. | Items without any owners at all are generally handled by those under "Default." However, in general, they could benefit from being their own strongly-owned modules at some point. | ||
{| border="1" cellpadding="4" | {| border="1" cellpadding="4" | ||
| Line 10: | Line 11: | ||
|- | |- | ||
|Default<br>(stuff under /mozilla/calendar not otherwise listed here) | |Default<br>(stuff under /mozilla/calendar not otherwise listed here) | ||
|mvl<sup>*</sup> | |philipp | ||
|daniel, ctalbert, mvl<sup>*</sup>, lilmatt<sup>*</sup> | |||
|- | |- | ||
|colspan="3"|'''Applications/Extensions''' | |colspan="3"|'''Applications/Extensions''' | ||
| Line 17: | Line 18: | ||
|Sunbird (calendar/sunbird) | |Sunbird (calendar/sunbird) | ||
|mvl<sup>*</sup> | |mvl<sup>*</sup> | ||
| | |ssitter, ctalbert, lilmatt<sup>*</sup>, daniel<sup>*</sup> | ||
|- | |- | ||
|Lightning (calendar/lightning) | |Lightning (calendar/lightning) | ||
|mvl<sup>*</sup> | |philipp | ||
|daniel, ctalbert, mvl<sup>*</sup>, lilmatt<sup>*</sup> | |||
|- | |- | ||
|colspan="3"|'''Components''' | |colspan="3"|'''Components''' | ||
|- | |- | ||
|Accessibility | |Accessibility | ||
| | |philipp | ||
| | | | ||
|- | |- | ||
|Alarms | |Alarms | ||
| | |philipp | ||
|daniel | |daniel | ||
|- | |- | ||
|Art & Branding | |Art & Branding | ||
|MarkCarson | |MarkCarson | ||
|lilmatt<sup>*</sup> | |||
|- | |- | ||
|Calendar Manager | |Calendar Manager | ||
|mvl<sup>*</sup> | |philipp | ||
|daniel, ctalbert, mvl<sup>*</sup> | |||
|- | |- | ||
|Import/Export (calendar/import-export) | |Import/Export (calendar/import-export) | ||
|mvl<sup>*</sup> | |mvl<sup>*</sup> | ||
| | | | ||
|- | |- | ||
|Installer (calendar/installer) | |Installer (calendar/installer) | ||
| | | | ||
|rob_strong | |mschroeder, ctalbert, rob_strong, lilmatt<sup>*</sup> | ||
|- | |- | ||
|i18n/l10n (calendar/locales) | |i18n/l10n (calendar/locales) | ||
| | |sipaq | ||
| | |lilmatt<sup>*</sup> | ||
|- | |- | ||
|libical (calendar/libical) | |libical (calendar/libical) | ||
|mvl<sup>*</sup> | |daniel | ||
|philipp, mvl<sup>*</sup> | |||
|- | |- | ||
|Recurrence | |Recurrence | ||
|daniel | |daniel | ||
|mvl<sup>*</sup> | |philipp, ctalbert, mvl<sup>*</sup> | ||
|- | |- | ||
|Timezones | |Timezones | ||
| | |daniel | ||
|mvl<sup>*</sup>, lilmatt | |ctalbert, mvl<sup>*</sup>, lilmatt<sup>*</sup> | ||
|- | |- | ||
|Views & Dialogs (calendar/{sunbird/}base/content) | |Views & Dialogs (calendar/{sunbird/}base/content) | ||
|philipp | |philipp | ||
|berend, mschroeder | |||
|- | |- | ||
|iTIP | |iTIP | ||
| | |daniel | ||
|lilmatt | |philipp, ctalbert, lilmatt<sup>*</sup> | ||
|- | |- | ||
|Printing | |Printing | ||
| | | | ||
|daniel | |daniel, lilmatt<sup>*</sup> | ||
|- | |- | ||
|Website | |Website | ||
| | |tobbi | ||
|lilmatt | |TMZ, lilmatt<sup>*</sup> | ||
|- | |- | ||
|User Interface (UI) | |User Interface (UI) | ||
| | |clarkbw | ||
| | |christian<sup>*</sup> | ||
|- | |- | ||
|colspan="3"|'''Providers''' | |colspan="3"|'''Providers''' | ||
| Line 89: | Line 90: | ||
|CalDAV | |CalDAV | ||
|bbrowning | |bbrowning | ||
|lilmatt | |philipp, daniel, lilmatt<sup>*</sup> | ||
|- | |- | ||
|Composite | |Composite | ||
|mvl<sup>*</sup> | |philipp | ||
|daniel, mvl<sup>*</sup> | |||
|- | |- | ||
|ICS | |ICS | ||
|mvl<sup>*</sup> | |daniel | ||
|philipp, mvl<sup>*</sup>, sebo | |||
|- | |- | ||
|Memory | |Memory | ||
|daniel | |daniel | ||
|philipp, mvl<sup>*</sup>, sebo | |||
|- | |- | ||
|Storage | |Storage | ||
|daniel | |daniel | ||
|philipp, mvl<sup>*</sup>, sebo | |||
|- | |- | ||
|WCAP | |WCAP | ||
|daniel | |daniel | ||
| | |philipp | ||
|- | |- | ||
|GDATA | |GDATA | ||
| Line 118: | Line 119: | ||
|- | |- | ||
|Linux | |Linux | ||
| | |philipp | ||
|daniel | |daniel | ||
|- | |- | ||
|Mac OS X | |Mac OS X | ||
| | |daniel | ||
| | |ctalbert, lilmatt<sup>*</sup> | ||
|- | |- | ||
|Windows | |Windows | ||
|ctalbert | |ctalbert | ||
| | | | ||
|} | |} | ||
<sup>*</sup> | <sup>*</sup> Backup reviewer. Assign to them rarely. | ||
If the bugzilla id is ":<name>", then bugzilla will expand it automatically. Just enter it into the reviewer field. | |||
{| border="0" | {| border="0" | ||
| Line 142: | Line 145: | ||
|bbrowning | |bbrowning | ||
|browning at uwalumni dot com | |browning at uwalumni dot com | ||
|- | |||
|berend | |||
|<var>:berend</var> | |||
|- | |- | ||
|christian | |christian | ||
| | |chris.j.bugzilla at googlemail dot com | ||
|- | |- | ||
|ctalbert | |ctalbert | ||
|ctalbert | |<var>:ctalbert</var> | ||
|- | |- | ||
|daniel | |daniel | ||
| | |<var>:dbo</var> | ||
|- | |- | ||
|lilmatt | |lilmatt | ||
| Line 157: | Line 163: | ||
|MarkCarson | |MarkCarson | ||
|mark at markcarson dot com | |mark at markcarson dot com | ||
|- | |||
|clarkbw | |||
|clarkbw at gnome dot org | |||
|} | |} | ||
| | | | ||
| Line 163: | Line 172: | ||
!Bugzilla ID | !Bugzilla ID | ||
|- | |- | ||
| | |mschroeder | ||
| | |<var>:mschroeder</var> | ||
|- | |- | ||
|mvl | |mvl | ||
| Line 170: | Line 179: | ||
|- | |- | ||
|philipp | |philipp | ||
| | |<var>:Fallen</var> | ||
|- | |- | ||
|rob_strong | |rob_strong | ||
|robert.bugzilla at gmail dot com | |robert.bugzilla at gmail dot com | ||
|- | |||
|sebo | |||
|sebo dot moz at googlemail dot com | |||
|- | |- | ||
|sipaq | |sipaq | ||
| | |<var>:sipaq</var> | ||
|- | |- | ||
|ssitter | |ssitter | ||
|ssitter | |<var>ssitter</var> | ||
|- | |||
|TMZ | |||
|<var>:TMZ</var> | |||
|- | |||
|Tobbi | |||
|<var>:Tobbi</var> | |||
|} | |} | ||
|} | |} | ||
=== Request Queue === | |||
When multiple reviewers come in question, you should check the [https://bugzilla.mozilla.org/request.cgi?action=queue&requester=&product=Calendar&type=review&requestee=&component=&group=requestee Calendar Request Queue] to see which reviewer has the smallest queue. | |||
=== Review Process === | === Review Process === | ||
Calendar does not require super-review. The only review is needed is the review of a module owner or peer of the involved module. (If a patch spans multiple modules, you need review from an owner or peer from each module. That sounds problematic, but due to the huge overlap in owners and peers, in most cases it can be done by one person.) | Calendar does not require super-review. The only review is needed is the review of a module owner or peer of the involved module. (If a patch spans multiple modules, you need review from an owner or peer from each module. That sounds problematic, but due to the huge overlap in owners and peers, in most cases it can be done by one person.) | ||
If the module owner and their peers are all busy and don't have time to do a complete review, you can ask someone else to review the code. This must be 'known hacker', somebody who has done calendar work himself, but does not need to be a module owner or peer. If you have code review, you can ask a owner or peer of the module your code touches to give module-owner-approval. Giving this approval only means that the code is good from a high level point of view. Because this is a lot less work, it should be easier to get this approval. | If the module owner and their peers are all busy and don't have time to do a complete review, you can ask someone else to review the code. This must be 'known hacker', somebody who has done calendar work himself, but does not need to be a module owner or peer. If you have code review, you can ask a owner or peer of the module your code touches to give module-owner-approval. Giving this approval only means that the code is good from a high level point of view. Because this is a lot less work, it should be easier to get this approval. | ||
Use the Bugzilla ID to identify the reviewer in the Requestee field of the Attachment form. | |||
[[category:calendar|Module Ownership]] | |||
Latest revision as of 18:17, 20 April 2020
This is a non-current list of owners and peers of the calendar modules. A more current list is here. Don't edit this file. Just don't.
Items without any owners at all are generally handled by those under "Default." However, in general, they could benefit from being their own strongly-owned modules at some point.
| Module | Owner | Peers (alpha order) |
|---|---|---|
| Default (stuff under /mozilla/calendar not otherwise listed here) |
philipp | daniel, ctalbert, mvl*, lilmatt* |
| Applications/Extensions | ||
| Sunbird (calendar/sunbird) | mvl* | ssitter, ctalbert, lilmatt*, daniel* |
| Lightning (calendar/lightning) | philipp | daniel, ctalbert, mvl*, lilmatt* |
| Components | ||
| Accessibility | philipp | |
| Alarms | philipp | daniel |
| Art & Branding | MarkCarson | lilmatt* |
| Calendar Manager | philipp | daniel, ctalbert, mvl* |
| Import/Export (calendar/import-export) | mvl* | |
| Installer (calendar/installer) | mschroeder, ctalbert, rob_strong, lilmatt* | |
| i18n/l10n (calendar/locales) | sipaq | lilmatt* |
| libical (calendar/libical) | daniel | philipp, mvl* |
| Recurrence | daniel | philipp, ctalbert, mvl* |
| Timezones | daniel | ctalbert, mvl*, lilmatt* |
| Views & Dialogs (calendar/{sunbird/}base/content) | philipp | berend, mschroeder |
| iTIP | daniel | philipp, ctalbert, lilmatt* |
| Printing | daniel, lilmatt* | |
| Website | tobbi | TMZ, lilmatt* |
| User Interface (UI) | clarkbw | christian* |
| Providers | ||
| CalDAV | bbrowning | philipp, daniel, lilmatt* |
| Composite | philipp | daniel, mvl* |
| ICS | daniel | philipp, mvl*, sebo |
| Memory | daniel | philipp, mvl*, sebo |
| Storage | daniel | philipp, mvl*, sebo |
| WCAP | daniel | philipp |
| GDATA | philipp | daniel |
| Tier-1 platform-specific stuff | ||
| Linux | philipp | daniel |
| Mac OS X | daniel | ctalbert, lilmatt* |
| Windows | ctalbert | |
* Backup reviewer. Assign to them rarely.
If the bugzilla id is ":<name>", then bugzilla will expand it automatically. Just enter it into the reviewer field.
| Legend | |||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
| ||||||||||||||||||||||||||||||||||||||
Request Queue
When multiple reviewers come in question, you should check the Calendar Request Queue to see which reviewer has the smallest queue.
Review Process
Calendar does not require super-review. The only review is needed is the review of a module owner or peer of the involved module. (If a patch spans multiple modules, you need review from an owner or peer from each module. That sounds problematic, but due to the huge overlap in owners and peers, in most cases it can be done by one person.)
If the module owner and their peers are all busy and don't have time to do a complete review, you can ask someone else to review the code. This must be 'known hacker', somebody who has done calendar work himself, but does not need to be a module owner or peer. If you have code review, you can ask a owner or peer of the module your code touches to give module-owner-approval. Giving this approval only means that the code is good from a high level point of view. Because this is a lot less work, it should be easier to get this approval.
Use the Bugzilla ID to identify the reviewer in the Requestee field of the Attachment form.