3,267
edits
Changes
Development Team/Code Review (view source)
Revision as of 19:57, 16 December 2008
, 19:57, 16 December 2008→Patch submission
== Patch submission ==
== Patch submission ==
* If your patch addresses an issue which is not registered in [http://dev.laptop.org/query trac], please open a new ticket for it.
* If your patch addresses an issue which is not registered in [http://dev.sugarlabs.org the Sugar Labs trac instance], please open a new ticket for it.
* Attach the patch to the ticket
* Attach the patch to the ticket
* Add the keyword 'r?' to call the attention of reviewers.
* Add the keyword 'r?' to call the attention of reviewers.
Click on Browse, Read, Pippy icons in the homepage and make sure all of them starts correctly.
Click on Browse, Read, Pippy icons in the homepage and make sure all of them starts correctly.
If your patch is a new feature and reasonably big, you may prefer to submit it for review to the Sugar [http://lists.laptop.org/listinfo/sugar mailing list]. If you do please add the review keyword to the ticket anyway and reference the mailing list thread.
If your patch is a new feature and reasonably big, you may prefer to submit it for review to the Sugar-devel [http://lists.sugarlabs.org/listinfo/sugar-devel mailing list]. If you do please add the review keyword to the ticket anyway and reference the mailing list thread.
In order to make it easy for the reviewer please:
In order to make it easy for the reviewer please: