[MERGE] Improved review process docs and separate out architectural overview
Vincent Ladeuil
v.ladeuil+lp at free.fr
Thu Sep 4 09:58:12 BST 2008
>>>>> "martin" == Martin Pool <mbp at sourcefrog.net> writes:
<snip/>
martin> === modified file 'doc/developers/HACKING.txt'
martin> --- doc/developers/HACKING.txt 2008-08-20 02:07:36 +0000
martin> +++ doc/developers/HACKING.txt 2008-09-04 04:55:36 +0000
<snip/>
martin> +Checklist for reviewers
martin> +=======================
martin> +
martin> +* Do you understand what the code's doing and why?
martin> +
* Does the change meet the requirements listed above (test coverage,
design clarity, improvements, etc).
<snip/>
martin> -
martin> -TODO: Incorporate John Arbash Meinel's detailed email to Ian C on the
martin> -numerous ways of setting up integration branches.
Was this deletion intended ? If yes, do you still have a copy of
this mail ? ;)
BB:tweak
Vincent
More information about the bazaar
mailing list