Bazaar

Bazaar

 




Wiki Tools

  • Find Page
  • Recent Changes
  • Page History
  • Attachments

Differences between revisions 33 and 71 (spanning 38 versions)
Revision 33 as of 2010-06-07 10:16:41
Size: 3526
Editor: garyvdm
Comment:
Revision 71 as of 2012-03-20 08:24:58
Size: 3889
Editor: mbp
Comment:
Deletions are marked like this. Additions are marked like this.
Line 3: Line 3:
The current pilot is listed in the IRC channel topic of ``#bzr`` on ``irc.freenode.or`` for easy reference.  The current pilot is listed in the IRC channel topic of ``#bzr`` on ``irc.freenode.or`` for easy reference.

The word ''pilot'' is in the sense of a [[http://en.wikipedia.org/wiki/Maritime_pilot|maritime pilot]]: we help patches come through congested waters safely in to harbor.
Line 7: Line 9:
 * 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.  * '''The main thing to watch is the [[https://code.launchpad.net/bzr/+activereviews|bzr active reviews page]] in Launchpad'''
Line 9: Line 11:
 * 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.  * 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.
Line 11: Line 13:
 * You're not obliged to deal with all the open patches. We appreciate what you do do.  * '''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.
Line 15: Line 19:
 * 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 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'''.
Line 21: Line 25:
 * 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.  * 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.
Line 25: Line 29:
 * 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.  * 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.
Line 27: Line 31:
 * You can also keep an eye on [[https://code.edge.launchpad.net/bzr/+merges?field.status=WORK_IN_PROGRESS&field.status-empty-marker=1|work in progress merges]] and [[https://bugs.edge.launchpad.net/bzr/+bugs?field.searchtext=&orderby=-date_last_updated&search=Search&field.status%3Alist=INPROGRESS&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_supervisor=&field.bug_commenter=&field.subscriber=&field.omit_dupes.used=&field.omit_dupes=on&field.has_patch.used=&field.has_cve.used=&field.tag=&field.tags_combinator=ANY|in progress bugs]]: these don't currently need review but do represent unfinished work.  * You can also keep an eye on [[https://code.launchpad.net/bzr/+merges?field.status=WORK_IN_PROGRESS&field.status-empty-marker=1|work in progress merges]] and [[https://bugs.launchpad.net/bzr/+bugs?field.searchtext=&orderby=-date_last_updated&search=Search&field.status%3Alist=INPROGRESS&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_supervisor=&field.bug_commenter=&field.subscriber=&field.omit_dupes.used=&field.omit_dupes=on&field.has_patch.used=&field.has_cve.used=&field.tag=&field.tags_combinator=ANY|in progress bugs]]: these don't currently need review but do represent unfinished work.
Line 29: Line 33:
 * The recommended way to send approved patches to PQM is using the {{{feed-pqm}}} script from https://launchpad.net/hydrazine/   * Also have a look at [[https://bugs.launchpad.net/bzr/+bugs?search=Search&field.status=New|new untriaged bugs]].

* The recommended way to send approved patches to PQM is using the {{{feed-pqm}}} script from https://launchpad.net/hydrazine/
Line 34: Line 40:

|| 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 || vila ||
|| 21 Jun w25 || lifeless ||
|| 28 Jun w26 || jam ||
|| 5 Jul w27 || poolie ||
|| 12 Jul w28 || Sprinting, GaryvdM - Dependent on work commitments ||
|| 15 Jan w03 || vila ||
|| 23 Jan w04 || jelmer ||
|| 30 Jan w05 || gz ||
|| 6 Feb w06 || poolie ||
|| 13 Feb w07 || vila ||
|| 20 Feb w08 || jelmer ||
|| 27 Feb w09 || gz ||
|| 19 Mar w12 || poolie ||
|| 26 Mar w13 || vila ||
|| 2 Apr w14 || jelmer ||
|| 9 Apr w15 || gz ||

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.

The word pilot is in the sense of a maritime pilot: we help patches come through congested waters safely in to harbor.

Tips for pilots:

  • The main thing to watch is the bzr active reviews page in Launchpad

  • 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.
  • 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.

  • Also have a look at new untriaged bugs.

  • 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

15 Jan w03

vila

23 Jan w04

jelmer

30 Jan w05

gz

6 Feb w06

poolie

13 Feb w07

vila

20 Feb w08

jelmer

27 Feb w09

gz

19 Mar w12

poolie

26 Mar w13

vila

2 Apr w14

jelmer

9 Apr w15

gz

See also: