/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

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

  • Committer: Jelmer Vernooij
  • Date: 2017-11-11 15:10:19 UTC
  • mfrom: (6803.1.2 devdocs)
  • Revision ID: jelmer@jelmer.uk-20171111151019-mlyo9bu1wi46rzld
Merge lp:~jelmer/brz/devdocs.

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/developers/code-review.txt
3
3
 
4
4
All non-trivial code changes coming in to Bazaar are reviewed by someone else.
5
5
 
6
 
Anyone is welcome to review any patch.  You don't need to have a full 
 
6
Anyone is welcome to review any patch.  You don't need to have a full
7
7
understanding of the codebase to find problems in the code, the documentation,
8
8
or the concept of the patch.
9
9
 
11
11
developer, and changes from other people are reviewed by two core
12
12
developers.  Use intelligent discretion about whether the patch is trivial.
13
13
 
14
 
No one likes their merge requests sitting in a queue going nowhere: this 
15
 
is pure waste.  We prioritize reviewing existing proposals.  
16
 
Canonical dedicates some staff time to providing prompt helpful reviews.  
 
14
No one likes their merge requests sitting in a queue going nowhere: this
 
15
is pure waste.  We prioritize reviewing existing proposals.
 
16
Canonical dedicates some staff time to providing prompt helpful reviews.
17
17
(See <http://wiki.bazaar.canonical.com/PatchPilot/>.)
18
18
 
19
 
From late 2009 on, we do all our code reviews through Launchpad's 
 
19
From late 2009 on, we do all our code reviews through Launchpad's
20
20
merge proposal interface.
21
21
 
22
22
 
85
85
Anyone can propose or comment on a merge proposal just by creating a
86
86
Launchpad account.
87
87
 
88
 
From <https://code.launchpad.net/bzr/+activereviews> you can see all
 
88
From <https://code.launchpad.net/brz/+activereviews> you can see all
89
89
currently active reviews, and choose one to comment on.  This page also
90
90
shows proposals that are now approved and should be merged by someone with
91
91
PQM access.
100
100
 
101
101
Once a merge proposal is approved and finished, it's sent to PQM (the patch
102
102
queue manager) which will automatically test and integrate it.  The recommended
103
 
way to start this off is by running the ``feed-pqm`` script from 
 
103
way to start this off is by running the ``feed-pqm`` script from
104
104
<https://launchpad.net/hydrazine/>.