Open main menu
Home
Random
Log in
Settings
About Sugar Labs
Disclaimers
Sugar Labs
Search
Changes
← Older edit
Newer edit →
Development Team/Code Review
(view source)
Revision as of 06:01, 28 July 2008
No change in size
,
06:01, 28 July 2008
→Reviewer guidelines
Line 57:
Line 57:
* For complex patches you might want to start with an overall design of the patch.
* For complex patches you might want to start with an overall design of the patch.
−
* Make sure the
submitted
provided a testcase before approving the patch
+
* Make sure the
submitter
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.
* Change the r? keyword to r+ if the patch is ok to be pushed.
Bernie
Bureaucrats
,
Check users
,
Administrators
1,784
edits