Development Team/Code Review: Difference between revisions

Line 40: Line 40:


* 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.laptop.org/query trac], please open a new ticket for it.
* Make sure that the ticket number is referenced in the commit comment and attach the patch to the ticket. Also, * Add the keyword 'r?' to call the attention of reviewers.
* Attach the patch to the ticket
* Add the keyword 'r?' to call the attention of reviewers.
* Add a testcase in a ticket comment. You need to mark with |TestCase|. For example:
* Add a testcase in a ticket comment. You need to mark with |TestCase|. For example:
  |TestCase|
  |TestCase|