зеркало из https://github.com/mozilla/gecko-dev.git
121 строка
5.7 KiB
ReStructuredText
121 строка
5.7 KiB
ReStructuredText
Getting reviews
|
|
===============
|
|
|
|
|
|
Thorough code reviews are one of Mozilla's ways of ensuring code quality.
|
|
Every patch must be reviewed by the module owner of the code, or one of their designated peers.
|
|
|
|
To request a review, you will need to specify a review group (starts with #). If there is not, you should select one or more usernames either when you submit the patch, or afterward in the UI.
|
|
If you have a mentor, the mentor can usually either also review or find a suitable reviewer on your behalf.
|
|
|
|
For example, the syntax to request review from a group should be:
|
|
|
|
.. code-block::
|
|
|
|
Bug xxxx - explain what you are doing and why r?#group-name
|
|
|
|
or
|
|
|
|
Bug xxxx - explain what you are doing and why r?developer-nickname
|
|
|
|
Getting attention: If a reviewer doesn't respond within a week, or so of the review request:
|
|
|
|
* Contact the reviewer directly (either via e-mail or on Matrix).
|
|
* Join developers on `Mozilla's Matrix server <https://chat.mozilla.org>`_, and ask if anyone knows why a review may be delayed. Please link to the bug too.
|
|
* If the review is still not addressed, mail the reviewer directly, asking if/when they'll have time to review the patch, or might otherwise be able to review it.
|
|
|
|
Review groups
|
|
-------------
|
|
|
|
|
|
.. list-table::
|
|
:header-rows: 1
|
|
|
|
* - Name
|
|
- Owns
|
|
- Members
|
|
* - #anti-tracking or #anti-tracking-reviewers
|
|
- `Anti-Tracking <https://wiki.mozilla.org/Modules/All#Anti-Tracking>`__
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/157/>`__
|
|
* - #build or #firefox-build-system-reviewers
|
|
- The configure & build system
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/20/>`__
|
|
* - #dom-workers-and-storage-reviewers
|
|
- DOM Workers & Storage
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/115/>`__
|
|
* - #devtools-reviewers
|
|
- Firefox DevTools
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/153/>`__
|
|
* - #fluent-reviewers
|
|
- Changes to Fluent (FTL) files (translation).
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/105/>`__
|
|
* - #firefox-source-docs-reviewers
|
|
- Documentation files and its build
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/118/>`__
|
|
* - #firefox-ux-team
|
|
- User experience (UX)
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/91/>`__
|
|
* - #firefox-svg-reviewers
|
|
- SVG-related changes
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/97/>`__
|
|
* - #geckoview-reviewers
|
|
- Changes to GeckoView
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/92/>`__
|
|
* - #gfx-reviewers
|
|
- Changes to Graphics code
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/122/>`__
|
|
* - #intermittent-reviewers
|
|
- Test manifest changes
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/110/>`__
|
|
* - #layout-reviewers
|
|
- Layout changes.
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/126/>`__
|
|
* - #linter-reviewers
|
|
- tools/lint/*
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/119/>`__
|
|
* - #mozbase
|
|
- Changes to Mozbase
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/113/>`__
|
|
* - #mozbase-rust
|
|
- Changes to Mozbase in Rust
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/114/>`__
|
|
* - #nss-reviewers
|
|
- Changes to Network Security Services (NSS)
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/156/>`__
|
|
* - #perftest-reviewers
|
|
- Perf Tests
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/102/>`__
|
|
* - #permissions or #permissions-reviewers
|
|
- `Permissions <https://wiki.mozilla.org/Modules/All#Permissions>`__
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/158/>`__
|
|
* - #platform-i18n-reviewers
|
|
- Platform Internationalization
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/150/>`__
|
|
* - #preferences-reviewers
|
|
- Firefox for Desktop Preferences (Options) user interface
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/132/>`__
|
|
* - #remote-debugging-reviewers
|
|
- Remote Debugging UI & tools
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/108/>`__
|
|
* - #static-analysis-reviewers
|
|
- Changes related to Static Analysis
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/120/>`__
|
|
* - #style or #firefox-style-system-reviewers
|
|
- Firefox style system (servo, layout/style).
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/90/>`__
|
|
* - #webcompat-reviewers
|
|
- System addons maintained by the Web Compatibility team
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/124/>`__
|
|
* - #webdriver-reviewers
|
|
- Marionette and geckodriver (including MozBase Rust), and Remote Protocol with WebDriver BiDi, and CDP.
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/103/>`__
|
|
* - #webidl
|
|
- Changes related to WebIDL
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/112/>`__
|
|
* - #xpcom-reviewers
|
|
- Changes related to XPCOM
|
|
- `Member list <https://phabricator.services.mozilla.com/project/members/125/>`__
|
|
|
|
To create a new group, fill a `new bug in Conduit::Administration <https://bugzilla.mozilla.org/enter_bug.cgi?product=Conduit&component=Administration>`__.
|
|
See `bug 1613306 <https://bugzilla.mozilla.org/show_bug.cgi?id=1613306>`__ as example.
|