Diff for "ReviewerMeetingAgenda"

Not logged in - Log In / Register

Differences between revisions 85 and 110 (spanning 25 versions)
Revision 85 as of 2009-03-21 17:58:39
Size: 7146
Comment:
Revision 110 as of 2009-06-16 15:25:40
Size: 7389
Editor: barry
Comment:
Deletions are marked like this. Additions are marked like this.
Line 13: Line 13:
 * AMEU '''Wednesday 08-Apr-2009 1500 UTC - 1545 UTC'''
 * Asia``Pac '''Wednesday 08-Apr-2009 2230 UTC - 2315 UTC'''
 * AMEU '''Wednesday 17-Jun-2009 1500 UTC - 1545 UTC'''
 * Asia``Pac '''Wednesday 17-Jun-2009 2230 UTC - 2315 UTC'''
Line 18: Line 18:
'''NOTE: NO MEETINGS 25-MAR OR 01-APR 2009'''
Line 24: Line 22:
 * Peanut gallery (anything not on the agenda)
Line 27: Line 24:
 * Dogtests (intellectronica)
 * Code must be used once or more to land (excluding tests) (intellectronica)
 * Peanut gallery (anything not on the agenda)
   * naming of unittest methods (Camel``Case or under_scores) (henninge)
   * order of assert parameters (expected, observed) (hennige)
 * Driving maintainable Javascript code (intellectronica)
  * Modularization and unit testing are strongly encouraged
  * Refactoring code after a UI review is often a good idea
  * The upper limit for JS diffs is 1600 lines (because javascript is so verbose)
Line 34: Line 36:
 * your name, date
Line 38: Line 38:
 * 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
 * gary_poster to take importfascist and rSP() discussion to ml
Line 52: Line 49:
 * [[ReviewerMeeting20090610]]
 * [[ReviewerMeeting20090603]]
 * [[ReviewerMeeting20090513]]
 * [[ReviewerMeeting20090506]]
 * [[ReviewerMeeting20090429]]
 * [[ReviewerMeeting20090422]]
 * [[ReviewerMeeting20090415]]
 * [[ReviewerMeeting20090408]]

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 17-Jun-2009 1500 UTC - 1545 UTC

  • AsiaPac Wednesday 17-Jun-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.

Agenda

  • Roll call
  • Action items
  • Mentoring update
  • Peanut gallery (anything not on the agenda)
    • naming of unittest methods (CamelCase or under_scores) (henninge)

    • order of assert parameters (expected, observed) (hennige)
  • Driving maintainable Javascript code (intellectronica)
    • Modularization and unit testing are strongly encouraged
    • Refactoring code after a UI review is often a good idea
    • The upper limit for JS diffs is 1600 lines (because javascript is so verbose)

Apologies

Please include the date for which you will be unavailable.

Outstanding Actions

  • gary_poster to take importfascist and rSP() discussion to ml

Notes

Proposed items

  • your topic, your name

Rejected items

Summaries of previous meetings

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