Development Team/Code Review: Difference between revisions

Marcopg (talk | contribs)
Erikos (talk | contribs)
Line 41: Line 41:
* 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.
* 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.
* 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|