/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: 2020-03-22 01:35:14 UTC
  • mfrom: (7490.7.6 work)
  • mto: This revision was merged to the branch mainline in revision 7499.
  • Revision ID: jelmer@jelmer.uk-20200322013514-7vw1ntwho04rcuj3
merge lp:brz/3.1.

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/developers/code-review.txt
1
 
Reviewing proposed changes to Bazaar
 
1
Reviewing proposed changes to Breezy
2
2
####################################
3
3
 
4
 
All non-trivial code changes coming in to Bazaar are reviewed by someone else.
 
4
All non-trivial code changes coming in to Breezy are reviewed by someone else.
5
5
 
6
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,
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.
16
 
Canonical dedicates some staff time to providing prompt helpful reviews.
17
 
(See <http://wiki.bazaar.canonical.com/PatchPilot/>.)
18
16
 
19
 
From late 2009 on, we do all our code reviews through Launchpad's
20
 
merge proposal interface.
 
17
We do all our code reviews through Launchpad's merge proposal interface.
21
18
 
22
19
 
23
20
Reviewing proposed changes
48
45
New things can easily be recorded in the bug tracker instead.
49
46
 
50
47
It's normally much easier to review several smaller patches than one large
51
 
one.  You might want to use ``bzr-loom`` to maintain threads of related
52
 
work, or submit a preparatory patch that will make your "real" change
53
 
easier.
 
48
one.  You might want to submit a preparatory patch that will make your "real"
 
49
change easier.
54
50
 
55
51
 
56
52
Checklist for reviewers
98
94
Landing approved changes
99
95
========================
100
96
 
101
 
Once a merge proposal is approved and finished, it's sent to PQM (the patch
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
104
 
<https://launchpad.net/hydrazine/>.
 
97
Once a merge proposal is approved and finished, it's marked as
 
98
Approved and picked up by the CI (https://ci.breezy-vcs.org/),
 
99
which will automatically test and integrate it.