From 6bf0a5cb5034a7e684dcc3500e841785237ce2dd Mon Sep 17 00:00:00 2001 From: Daniel Baumann Date: Sun, 7 Apr 2024 19:32:43 +0200 Subject: Adding upstream version 1:115.7.0. Signed-off-by: Daniel Baumann --- docs/contributing/reviews.rst | 145 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 145 insertions(+) create mode 100644 docs/contributing/reviews.rst (limited to 'docs/contributing/reviews.rst') diff --git a/docs/contributing/reviews.rst b/docs/contributing/reviews.rst new file mode 100644 index 0000000000..0cb0dfb59c --- /dev/null +++ b/docs/contributing/reviews.rst @@ -0,0 +1,145 @@ +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 `_, 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. + +For simple documentation changes, reviews are not required. + +For more information about the review process, see the :ref:`Code Review FAQ`. + +Review groups +------------- + + +.. list-table:: + :header-rows: 1 + + * - Name + - Owns + - Members + * - #anti-tracking + - `Core: Anti-Tracking `__ + - `Member list `__ + * - #build or #firefox-build-system-reviewers + - The configure & build system + - `Member list `__ + * - #cookies + - `Core: Cookies `__ + - `Member list `__ + * - #dom-workers-and-storage-reviewers + - DOM Workers & Storage + - `Member list `__ + * - #devtools-reviewers + - Firefox DevTools + - `Member list `__ + * - #fluent-reviewers + - Changes to Fluent (FTL) files (translation). + - `Member list `__ + * - #firefox-source-docs-reviewers + - Documentation files and its build + - `Member list `__ + * - #firefox-ux-team + - User experience (UX) + - `Member list `__ + * - #firefox-svg-reviewers + - SVG-related changes + - `Member list `__ + * - #geckoview-reviewers + - Changes to GeckoView + - `Member list `__ + * - #gfx-reviewers + - Changes to Graphics code + - `Member list `__ + * - #webgpu-reviewers + - Changes to WebGPU code + - `Member list `__ + * - #intermittent-reviewers + - Test manifest changes + - `Member list `__ + * - #layout-reviewers + - Layout changes. + - `Member list `__ + * - #linter-reviewers + - tools/lint/* + - `Member list `__ + * - #mac-reviewers + - Changes to Mac-specific code + - `Member list `__ + * - #mozbase + - Changes to Mozbase + - `Member list `__ + * - #mozbase-rust + - Changes to Mozbase in Rust + - `Member list `__ + * - #necko-reviewers + - Changes to network code (aka necko, aka netwerk) + - `Member list `__ + * - #nss-reviewers + - Changes to Network Security Services (NSS) + - `Member list `__ + * - #perftest-reviewers + - Perf Tests + - `Member list `__ + * - #permissions or #permissions-reviewers + - `Permissions `__ + - `Member list `__ + * - #platform-i18n-reviewers + - Platform Internationalization + - `Member list `__ + * - #preferences-reviewers + - Firefox for Desktop Preferences (Options) user interface + - `Member list `__ + * - #remote-debugging-reviewers + - Remote Debugging UI & tools + - `Member list `__ + * - #spidermonkey-reviewers + - SpiderMonkey JS/Wasm Engine + - `Member list `__ + * - #static-analysis-reviewers + - Changes related to Static Analysis + - `Member list `__ + * - #style or #firefox-style-system-reviewers + - Firefox style system (servo, layout/style). + - `Member list `__ + * - #webcompat-reviewers + - System addons maintained by the Web Compatibility team + - `Member list `__ + * - #webdriver-reviewers + - Marionette and geckodriver (including MozBase Rust), and Remote Protocol with WebDriver BiDi, and CDP. + - `Member list `__ + * - #webidl + - Changes related to WebIDL + - `Member list `__ + * - #xpcom-reviewers + - Changes related to XPCOM + - `Member list `__ + * - #media-playback-reviewers + - `Media playback `__ + - `Member list `__ + * - #cubeb-reviewers + - Changes related to cubeb, Gecko's audio input/output library and associated projects (audioipc, cubeb-rs, rust cubeb backends) + - `Member list `__ + +To create a new group, fill a `new bug in Conduit::Administration `__. +See `bug 1613306 `__ as example. -- cgit v1.2.3