Compare Revisions

Mailnews and Mail code review requirements

Revision 233975:

Revision 233975 by asutherland@asutherland.org on

Revision 88624:

Revision 88624 by asutherland@asutherland.org 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 233975
Revision 88624
n13    <h3>n13    <h3 id="Mailnews_and_Mail_review_rules">
n16    <h4>n16    <h4 id="Patches_affecting_Thunderbird_User_Experience_or_Inte
 >rfaces">
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. Bryan Clark (t>t an early stage in the patch development process. Blake Winton (
>he Thunderbird User Experience guy) is more than happy to ui-revi>the Thunderbird User Experience guy) is more than happy to ui-rev
>ew screenshots or outlines before getting towards the final patch>iew screenshots or outlines before getting towards the final patc
>.") }}>h.") }}
n26      <li>All patches that affect user experience or interfaces on26      <li>All patches that affect user experience or interfaces o
>n Thunderbird should have ui-review on those patches in addition >n Thunderbird should have ui-review on those patches in addition 
>to the reviews required below.<br>>to the reviews required below.
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>:clarkbw</code> as the requestee.>view to '?' and enter <code>:bwinton</code> as the requestee.
n33    <h4>n33    <h4 id="Rules_for_all_patches_affecting_mailnews/_and_mail/">
n48    <h3>n48    <h3 id="Unit_test_rules">
n56      <li>If an <a class="external" href="http://developer.mozilln56      <li>If an <a class="external" href="/en/docs/Mozilla_automa
>a.org/en/docs/Mozilla_automated_testing">automated test framework>ted_testing">automated test framework</a> is needed but is not ye
></a> is needed but is not yet available, the developer is encoura>t available, the developer is encouraged to write appropriate tes
>ged to write appropriate test code and commit it. A bug should be>t code and commit it. A bug should be filed on the needed test fr
> filed on the needed test framework. The <code>in-testsuite?</cod>amework. The <code>in-testsuite?</code> flag should be set on the
>e> flag should be set on the bug until the framework has been com> bug until the framework has been completed and the test code is 
>pleted and the test code is running automatically.>running automatically.
n61    <h3>n61    <h3 id='Rubber-Stamp_Approvals_for_Intermittently_Failing&nbs
 >p;("Orange")_Test_Fixes/Debugging'>
n75    <h4>n75    <h4 id="rs=simple-orange-fix">
n97    <h4>n97    <h4 id="rs=orange-debugging">
n117    <h3>n117    <h3 id="Mailnews_individuals_and_roles">
t120    <h3>t120    <h3 id="Mail_individuals_and_roles">

Back to History