647
edits
Changes
Development Team/Code Review (view source)
Revision as of 03:44, 12 August 2009
, 03:44, 12 August 2009→Proposal
* 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.
* 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
* Alternatively, you can add a link pointing to a cloned tree. Make sure it contains only the changes you propose for review.
* Add the keyword 'r?' to call the attention of reviewers.
* Add the keyword 'r?' to call the attention of reviewers.
* Add a testcase in a ticket comment. You need to mark it with |TestCase|. This adds better readability and our script that pulls together the test cases for each release is able to find it as well. For example:
* Add a testcase in a ticket comment. You need to mark it with |TestCase|. This adds better readability and our script that pulls together the test cases for each release is able to find it as well. For example: