Diff for "ReviewerMeetingAgenda"

Not logged in - Log In / Register

Differences between revisions 72 and 88 (spanning 16 versions)
Revision 72 as of 2009-02-26 00:12:09
Size: 7420
Editor: barry
Comment:
Revision 88 as of 2009-04-08 13:55:02
Size: 7194
Comment:
Deletions are marked like this. Additions are marked like this.
Line 13: Line 13:
 * Asia``Pac '''Wednesday 04-Mar-2009 2230 UTC - 2315 UTC'''
 * AMEU '''Wednesday 04-Mar-2009 1500 UTC - 1545 UTC'''
 * AMEU '''Wednesday 08-Apr-2009 1500 UTC - 1545 UTC'''
 * Asia``Pac '''Wednesday 08-Apr-2009 2230 UTC - 2315 UTC'''
Line 18: Line 18:
'''NOTE: NO MEETINGS 25-MAR OR 01-APR 2009'''
Line 22: Line 24:
 * Reviewers to enforce bug<->branch linking for all non-trivial branches (intellectronica)
 * Determine official preference, if any, of Storm usage: And(expr, expr) vs. expr & expr, Gary
 * Determine why _get_store uses protected spelling, and if we think it should, Gary
 * Merge proposals and cover letters (barry)
 * Using OCR for JavaScript reviews (mars)
Line 28: Line 25:
 * Action items
Line 29: Line 27:
 * Action items  * Code must be used once or more to land (excluding tests) (intellectronica)
 * Avoid refactoring both code and tests in the same branch (intellectronica)
Line 39: Line 38:
 * bigjools to take crack at helper functions for backpatching schemas to avoid circular imports
 * intellectronica to come back from sabbatical
 * flacoste to look into storm/sqlobject result set compatibility
 * gary_poster will check to see if there's a bug open for adding a hook to `bzr send`, and submit one if there isn't
Line 42: Line 41:
 * abentley to experiment on storm base class
 * barry to add `field_id` to coding guideline
 * barry to update guidelines to never call `_foo()` methods from outside a class
 * barry to add `pretty()` functions to reviewers docs
Line 57: Line 52:
 * [[ReviewerMeeting20090318]]
 * [[ReviewerMeeting20090311]]
 * [[ReviewerMeeting20090304]]
 * [[ReviewerMeeting20090225]]

Agenda of the next Launchpad review team meeting

Launchpad review team meetings are held on #launchpad-meeting on irc.freenode.net, and are chaired by BarryWarsaw for both the Asia/Pacific (AsiaPac) and Americas/EU (AMEU) time zone meetings. We use MootBot to record the meetings.

The purpose of the meetings is to coordinate Launchpad code reviews, and particularly to ensure that emerging issues with the review process are addressed - such as getting people to take on tough reviews, keeping the latency down, etc.. Blockage in reviews leads to blockage in Launchpad code landings.

The next Launchpad review team meeting will be held at this time:

  • AMEU Wednesday 08-Apr-2009 1500 UTC - 1545 UTC

  • AsiaPac Wednesday 08-Apr-2009 2230 UTC - 2315 UTC

You can add new items to the list of proposed items. If they don't make it onto this meetings agenda, they'll most likely get onto the next meeting's agenda instead.

NOTE: NO MEETINGS 25-MAR OR 01-APR 2009

Agenda

  • Roll call
  • Peanut gallery (anything not on the agenda)
  • Action items
  • Mentoring update
  • Code must be used once or more to land (excluding tests) (intellectronica)
  • Avoid refactoring both code and tests in the same branch (intellectronica)

Apologies

Please include the date for which you will be unavailable.

  • your name, date

Outstanding Actions

  • flacoste to look into storm/sqlobject result set compatibility
  • gary_poster will check to see if there's a bug open for adding a hook to bzr send, and submit one if there isn't

  • thumper to open bug on webservice pagetests globs problem

  • flacoste to work on API reviewer cheat sheet

Notes

Proposed items

  • your topic, your name

Rejected items

Summaries of previous meetings

ReviewerMeetingAgenda (last edited 2011-01-24 20:30:18 by bac)