← 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: Tarmac
  • Author(s): Vincent Ladeuil, Patch Queue Manager, Jelmer Vernooij
  • Date: 2017-01-17 16:20:41 UTC
  • mfrom: (6619.1.2 trunk)
  • Revision ID: tarmac-20170117162041-oo62uk1qsmgc9j31
Merge 2.7 into trunk including fixes for bugs #1622039, #1644003, #1579093 and #1645017. [r=vila]

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 release notes 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?