Development Team/Code Review: Difference between revisions

Erikos (talk | contribs)
Line 58: Line 58:
* Make sure the submitted provided a testcase before approving the patch
* Make sure the submitted provided a testcase before approving the patch
* Change the r? keyword to r- to indicate that the patch needs work.
* Change the r? keyword to r- to indicate that the patch needs work.
* Change the r? keyword to r+ if the patch is ok to be pushed.


== Push the patch ==
== Push the patch ==