Compare Revisions

Mailnews and Mail code review requirements

Revision 88624:

Revision 88624 by asutherland@asutherland.org on

Revision 467555:

Revision 467555 by tessarakt4 on

Title:
Mailnews and Mail code review requirements
Mailnews and Mail code review requirements
Slug:
Mailnews_and_Mail_code_review_requirements
Mailnews_and_Mail_code_review_requirements
Content:

Revision 88624
Revision 467555
n20      {{ Note("It is recommended that when working on bugs that an20      {{ Note("It is recommended that when working on bugs that a
>ffect user experience or interfaces, that ui-review is obtained a>ffect user experience or interfaces, that ui-review is obtained a
>t an early stage in the patch development process. Blake Winton (>t an early stage in the patch development process. Bryan Clark (t
>the Thunderbird User Experience guy) is more than happy to ui-rev>he Thunderbird User Experience guy) is more than happy to ui-revi
>iew screenshots or outlines before getting towards the final patc>ew screenshots or outlines before getting towards the final patch
>h.") }}>.") }}
n28          <li>To obtained a ui-review on an attachment, set ui-ren28          <li>To obtained a ui-review on an attachment, set ui-re
>view to '?' and enter <code>:bwinton</code> as the requestee.>view to '?' and enter <code>:clarkbw</code> as the requestee.
n33    <h4 id="Rules_for_all_patches_affecting_mailnews/_and_mail/">n33    <h4 id="Rules_for_all_patches_affecting_mailnews.2F_and_mail.
 >2F">
n45      <li>Super-review is required in certain situations:&nbsp;sin45      <li>Super-review is required in certain situations:&nbsp;si
>gnificant architectural refactoring, any change to any API, all c>gnificant architectural refactoring, any change to any API, all c
>hanges that affect how code modules interact. See the full super->hanges that affect how code modules interact. See the <a href="ht
>review policy for more details <em>(although this doesn't current>tp://www.mozilla.org/hacking/reviewers.html" title="http://www.mo
>ly mention comm-central, we have a <a class=" link-https" href="h>zilla.org/hacking/reviewers.html">full super-review policy</a> fo
>ttps://bugzilla.mozilla.org/show_bug.cgi?id=582229" title="https:>r more details.
>//bugzilla.mozilla.org/show_bug.cgi?id=582229">bug on file</a> to 
> get comm-central added)</em>. 
n56      <li>If an <a class="external" href="/en/docs/Mozilla_automan56      <li>If an <a class="external" href="http://developer.mozill
>ted_testing">automated test framework</a> is needed but is not ye>a.org/en/docs/Mozilla_automated_testing">automated test framework
>t available, the developer is encouraged to write appropriate tes></a> is needed but is not yet available, the developer is encoura
>t code and commit it. A bug should be filed on the needed test fr>ged to write appropriate test code and commit it. A bug should be
>amework. The <code>in-testsuite?</code> flag should be set on the> filed on the needed test framework. The <code>in-testsuite?</cod
> bug until the framework has been completed and the test code is >e> flag should be set on the bug until the framework has been com
>running automatically.>pleted and the test code is running automatically.
n61    <h3 id='Rubber-Stamp_Approvals_for_Intermittently_Failing&nbsn61    <h3 id="Rubber-Stamp_Approvals_for_Intermittently_Failing.C2.
>p;("Orange")_Test_Fixes/Debugging'>>A0(.22Orange.22)_Test_Fixes.2FDebugging">
n65      In order to make it easier to debug and fix tests that failn65      In order to make it easier to debug and fix tests that fail
> intermittently ("intermittent orange" tests), we have created th> intermittently ("intermittent orange" tests), we have created th
>e following two rubber-stamps based on <a class=" link-https" hre>e following two rubber-stamps based on <a class="link-https" href
>f="https://mail.mozilla.org/pipermail/tb-planning/2011-March/0008>="https://mail.mozilla.org/pipermail/tb-planning/2011-March/00089
>99.html" title="https://mail.mozilla.org/pipermail/tb-planning/20>9.html" title="https://mail.mozilla.org/pipermail/tb-planning/201
>11-March/000899.html">this proposal on the tb-planning mailing li>1-March/000899.html">this proposal on the tb-planning mailing lis
>st</a>.&nbsp; The procedure to use these is to be sure to:>t</a>.&nbsp; The procedure to use these is to be sure to:
n75    <h4 id="rs=simple-orange-fix">n75    <h4 id="rs.3Dsimple-orange-fix">
n97    <h4 id="rs=orange-debugging">n97    <h4 id="rs.3Dorange-debugging">
n118      <a class=" link-https" href="https://wiki.mozilla.org/Moduln118      <a class="link-https" href="https://wiki.mozilla.org/Module
>es/MailNews_Core" title="https://wiki.mozilla.org/Modules/MailNew>s/MailNews_Core" title="https://wiki.mozilla.org/Modules/MailNews
>s_Core">Mailnews individuals and roles</a>>_Core">Mailnews individuals and roles</a>
t121      <a class=" link-https" href="https://wiki.mozilla.org/Modult121      <a class="link-https" href="https://wiki.mozilla.org/Module
>es:Thunderbird" title="https://wiki.mozilla.org/Modules:Thunderbi>s:Thunderbird" title="https://wiki.mozilla.org/Modules:Thunderbir
>rd">Mail individuals and roles</a>>d">Mail individuals and roles</a>

Back to History