← 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: 2011-03-15 07:54:39 UTC
  • mfrom: (0.38.5 trunk)
  • mto: This revision was merged to the branch mainline in revision 5726.
  • Revision ID: andrew.bennetts@canonical.com-20110315075439-nzm293joz143cx0k
Merge bzr-changelog-merge plugin.

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 if the patch is trivial.
 
12
developers.  Use intelligent discretion about whether 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?
 
69
  appropriately documented in NEWS and/or in whats-new ?
70
70
 
71
 
* Does it meet the coding standards below?
 
71
* Does it meet the `coding standards <code-style.html>`_?
72
72
 
73
73
* If it changes the user-visible behaviour, does it update the help
74
74
  strings and user documentation?