Development Team/Code Review: Difference between revisions
m DevelopmentTeam/CodeReview moved to Development Team/CodeReview: deCamel casing |
m fix camelcase links |
||
| Line 58: | Line 58: | ||
* 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 submitter provided a testcase before approving the patch. | * Make sure the submitter provided a testcase before approving the patch. | ||
* Make sure that [[ | * Make sure that [[Development Team/API policy|API policy]] rules are respected. | ||
* Keep an eye on readability, we are roughly following Guido's [http://www.python.org/dev/peps/pep-0008 PEP 8] | * Keep an eye on readability, we are roughly following Guido's [http://www.python.org/dev/peps/pep-0008 PEP 8] | ||
* 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. | ||
| Line 91: | Line 91: | ||
[[Category: | [[Category:Development Team]] | ||