Calendar:Module Ownership: Difference between revisions
(l10n +sipaq) |
(move lilmatt to backup status) |
||
| Line 11: | 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> | |mvl<sup>*</sup> | ||
|lilmatt, daniel, mickey, ctalbert | |lilmatt<sup>*</sup>, daniel, mickey, ctalbert | ||
|- | |- | ||
|colspan="3"|'''Applications/Extensions''' | |colspan="3"|'''Applications/Extensions''' | ||
| Line 17: | Line 17: | ||
|Sunbird (calendar/sunbird) | |Sunbird (calendar/sunbird) | ||
|mvl<sup>*</sup> | |mvl<sup>*</sup> | ||
|daniel, lilmatt, ctalbert | |daniel, lilmatt<sup>*</sup>, ctalbert | ||
|- | |- | ||
|Lightning (calendar/lightning) | |Lightning (calendar/lightning) | ||
|mvl<sup>*</sup> | |mvl<sup>*</sup> | ||
|daniel, lilmatt, ctalbert | |daniel, lilmatt<sup>*</sup>, ctalbert | ||
|- | |- | ||
|colspan="3"|'''Components''' | |colspan="3"|'''Components''' | ||
| Line 34: | Line 34: | ||
|- | |- | ||
|Art & Branding | |Art & Branding | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|MarkCarson | |MarkCarson | ||
|- | |- | ||
| Line 46: | Line 46: | ||
|- | |- | ||
|Installer (calendar/installer) | |Installer (calendar/installer) | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|rob_strong, ctalbert | |rob_strong, ctalbert | ||
|- | |- | ||
|i18n/l10n (calendar/locales) | |i18n/l10n (calendar/locales) | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|sipaq | |sipaq | ||
|- | |- | ||
| Line 63: | Line 63: | ||
|Timezones | |Timezones | ||
|ctalbert | |ctalbert | ||
|mvl<sup>*</sup>, lilmatt, daniel | |mvl<sup>*</sup>, lilmatt<sup>*</sup>, daniel | ||
|- | |- | ||
|Views & Dialogs (calendar/{sunbird/}base/content) | |Views & Dialogs (calendar/{sunbird/}base/content) | ||
| Line 71: | Line 71: | ||
|iTIP | |iTIP | ||
|ctalbert | |ctalbert | ||
|lilmatt, daniel | |lilmatt<sup>*</sup>, daniel | ||
|- | |- | ||
|Printing | |Printing | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|daniel | |daniel | ||
|- | |- | ||
|Website | |Website | ||
|sipaq | |sipaq | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|- | |- | ||
|User Interface (UI) | |User Interface (UI) | ||
| Line 89: | Line 89: | ||
|CalDAV | |CalDAV | ||
|bbrowning | |bbrowning | ||
|lilmatt, daniel | |lilmatt<sup>*</sup>, daniel | ||
|- | |- | ||
|Composite | |Composite | ||
| Line 122: | Line 122: | ||
|- | |- | ||
|Mac OS X | |Mac OS X | ||
|lilmatt | |lilmatt<sup>*</sup> | ||
|ctalbert | |ctalbert | ||
|- | |- | ||
| Line 130: | Line 130: | ||
|} | |} | ||
<sup>*</sup> | <sup>*</sup> Backup reviewer. Assign to them rarely. | ||
{| border="0" | {| border="0" | ||
Revision as of 16:14, 8 August 2007
This is the current list of owners and peers of the calendar modules. 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) |
mvl* | lilmatt*, daniel, mickey, ctalbert |
| Applications/Extensions | ||
| Sunbird (calendar/sunbird) | mvl* | daniel, lilmatt*, ctalbert |
| Lightning (calendar/lightning) | mvl* | daniel, lilmatt*, ctalbert |
| Components | ||
| Accessibility | ? | ? |
| Alarms | ? | daniel, philipp |
| Art & Branding | lilmatt* | MarkCarson |
| Calendar Manager | mvl* | daniel, ctalbert |
| Import/Export (calendar/import-export) | mvl* | mickey |
| Installer (calendar/installer) | lilmatt* | rob_strong, ctalbert |
| i18n/l10n (calendar/locales) | lilmatt* | sipaq |
| libical (calendar/libical) | mvl* | daniel, mickey |
| Recurrence | daniel | mvl*, mickey, ctalbert |
| Timezones | ctalbert | mvl*, lilmatt*, daniel |
| Views & Dialogs (calendar/{sunbird/}base/content) | mickey | philipp |
| iTIP | ctalbert | lilmatt*, daniel |
| Printing | lilmatt* | daniel |
| Website | sipaq | lilmatt* |
| User Interface (UI) | christian | mvl* |
| Providers | ||
| CalDAV | bbrowning | lilmatt*, daniel |
| Composite | mvl* | daniel, mickey |
| ICS | mvl* | daniel, mickey |
| Memory | mvl* | daniel |
| Storage | mvl* | daniel |
| WCAP | daniel | ? |
| GDATA | philipp | daniel |
| Tier-1 platform-specific stuff | ||
| Linux | daniel | ? |
| Mac OS X | lilmatt* | ctalbert |
| Windows | ctalbert | ssitter, daniel |
* Backup reviewer. Assign to them rarely.
| Legend | |||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
| ||||||||||||||||||||||||||||
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.