[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