← 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: John Arbash Meinel
  • Date: 2011-05-11 11:35:28 UTC
  • mto: This revision was merged to the branch mainline in revision 5851.
  • Revision ID: john@arbash-meinel.com-20110511113528-qepibuwxicjrbb2h
Break compatibility with python <2.6.

This includes auditing the code for places where we were doing
explicit 'sys.version' checks and removing them as appropriate.

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?