Bazaar

Bazaar

 




Wiki Tools

  • Find Page
  • Recent Changes
  • Page History
  • Attachments

Differences between revisions 29 and 30
Revision 29 as of 2010-05-12 09:07:05
Size: 3850
Editor: mbp
Comment: add pp rota
Revision 30 as of 2010-05-14 13:00:22
Size: 3454
Editor: mbp
Comment: cleanups
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
PatchPilot is the Bazaar community scheme to ensure that patches are not ignored or lost in traffic.

We have a volunteer roster of committers willing to mentor and assist patches to be accepted into ``bzr``.

Our [[http://doc.bazaar-vcs.org/current/developers/HACKING.html|contribution guidelines]] can be daunting, and patch pilots help people navigate them.
PatchPilot is the Bazaar community scheme to ensure that patches are not ignored or lost in traffic. When people want to improve something in Bazaar, we help them get their patch up to the right standard to land it.
Line 8: Line 4:

Roster - just put your name in here to select a week. Remember that this is a volunteer program, so its success depends on your contributions.
Line 35: Line 29:
 * To submit a remote branch to PQM: `bzr pqm-submit --ignore-local --submit-branch=http://bazaar.launchpad.net/~bzr-pqm/bzr/bzr.dev --public-location=BRANCH_URL`. This requires `pqm_email` to be defined in the global or location configs, e.g. {{{
[http://bazaar.launchpad.net/~*/bzr/]
pqm_email = Canonical PQM <pqm@bazaar-vcs.org>
}}}
 * The recommended way to send approved patches to PQM is using the {{{feed-pqm}}} script from https://launchpad.net/hydrazine/
Line 54: Line 45:

See also:

 * [[http://doc.bazaar.canonical.com/bzr.dev/developers/code-review.html|Code review process]]

PatchPilot is the Bazaar community scheme to ensure that patches are not ignored or lost in traffic. When people want to improve something in Bazaar, we help them get their patch up to the right standard to land it.

The current pilot is listed in the IRC channel topic of #bzr on irc.freenode.or for easy reference.

Tips for pilots:

  • When you're piloting, put some concentrated effort into helping people have a good and satisfying experience contributing to Bazaar. Just how you do that is up to you.
  • Send a brief mail before and/or after your stint, to say what you're going to do or what you did, and how it worked out. If you have feedback on the review system or the process, speak up.
  • You're not obliged to deal with all the open patches. We appreciate what you do do.
  • You can prioritize whichever you think best achieves the goal of helping people enjoy getting things done in Bazaar. That might be the newest ones, neglected patches, easy patches, or those from new contributors.
  • You can also choose to pick up patches and get them in yourself, by writing tests or fixing things up, or to teach/ask/encourage the original submitter to do these things. Do try to at least explain to the contributor what you're going to do.
  • You can ask (cajole) others to do reviews.
  • You can keep working on patches you're involved with when your stint ends, or you can let the next pilot pick them up. Either way, communicate.
  • If someone's already been asked to do particular changes to get a patch landed, don't move the goalposts without a very good reason.
  • Check that the contributors have executed the Canonical contributor agreement. You can do this automatically using the scan-merge-proposals script from Hydrazine or just by checking their membership of ~contributor-agreement-canonical through the web ui.

  • It's good to get to a specific outcome, so that people know what to do next, and to make this clear in both the text and formal status of the merge proposal. That might be: needs X specific fixes, needs Y tests, needs another review, just needs to be merged.
  • You can also keep an eye on work in progress merges and in progress bugs: these don't currently need review but do represent unfinished work.

  • The recommended way to send approved patches to PQM is using the feed-pqm script from https://launchpad.net/hydrazine/

We started 14 November 2009.

Week

Pilot

3 May

Vincent Ladeuil

10 May w19

Sprinting, no patch pilot

17 May w20

jam

24 May w21

lifeless

31 May w22

poolie

7 Jun w23

spiv

14 Jun w24

jam

21 Jun w25

lifeless

28 Jun w26

poolie

5 Jul w27

spiv

See also: