← 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: Andrew Bennetts
  • Date: 2010-09-23 07:40:07 UTC
  • mfrom: (5050.17.24 2.2)
  • mto: This revision was merged to the branch mainline in revision 5440.
  • Revision ID: andrew.bennetts@canonical.com-20100923074007-bqk101393z4npbw0
Merge lp:bzr/2.2.

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 NEWS 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?