Development Team/Code Review: Difference between revisions
m moved Development Team/CodeReview to Development Team/Code Review: deCamel caseing |
|||
| Line 27: | Line 27: | ||
* 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: | ||