← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to doc/developers/code-review.txt

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-01 08:02:42 UTC
  • mfrom: (5390.3.3 faster-revert-593560)
  • Revision ID: pqm@pqm.ubuntu.com-20100901080242-esg62ody4frwmy66
(spiv) Avoid repeatedly calling self.target.all_file_ids() in
 InterTree.iter_changes. (Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/developers/code-review.txt
9
9
 
10
10
Normally changes by core contributors are reviewed by one other core
11
11
developer, and changes from other people are reviewed by two core
12
 
developers.  Use intelligent discretion about whether the patch is trivial.
 
12
developers.  Use intelligent discretion about whether if the patch is trivial.
13
13
 
14
14
No one likes their merge requests sitting in a queue going nowhere: this 
15
15
is pure waste.  We prioritize reviewing existing proposals.  
66
66
  blackbox (command-line level) and API-oriented tests?
67
67
 
68
68
* If this change will be visible to end users or API users, is it
69
 
  appropriately documented in release notes and/or in whats-new ?
 
69
  appropriately documented in NEWS?
70
70
 
71
 
* Does it meet the `coding standards <code-style.html>`_?
 
71
* Does it meet the coding standards below?
72
72
 
73
73
* If it changes the user-visible behaviour, does it update the help
74
74
  strings and user documentation?