Diff for "ReviewerMeetingAgenda"

Not logged in - Log In / Register

Differences between revisions 9 and 216 (spanning 207 versions)
Revision 9 as of 2008-11-26 14:45:08
Size: 6821
Editor: barry
Comment:
Revision 216 as of 2010-01-06 20:30:09
Size: 8667
Editor: bac
Comment:
Deletions are marked like this. Additions are marked like this.
Line 4: Line 4:
`irc.freenode.net`, and are chaired by BarryWarsaw for both the Asia/Pacific (Asia``Pac) and Americas/EU (AMEU) time zone meetings. We use [[https://wiki.ubuntu.com/ScribesTeam/MootBot|MootBot]] to record the meetings. `irc.freenode.net`, and are chaired by BradCrittenden for both the Asia/Pacific (Asia``Pac) and Americas/EU (AMEU) time zone meetings. We use [[https://wiki.ubuntu.com/ScribesTeam/MootBot|MootBot]] to record the meetings.
Line 13: Line 13:
 * Asia``Pac '''Tuesday 02-Dec-2008 0300 UTC - 0345 UTC'''
 * AMEU '''Wednesday 26-Nov-2008 1500 UTC - 1545 UTC'''
 * AMEU '''Wednesday 06-Jan-2010 1500 UTC - 1545 UTC'''
 * Asia``Pac '''Wednesday 06-Jan-2010 2100 UTC - 2145 UTC'''
Line 22: Line 22:
 * Merge proposal bugs
   * 245257 - 4 (diffs/bundles)
   * 202000 - 3 (complete diffs)
   * 300462 - 2 (better follow up diffs)
   * 297547 - 2 (CC someone via email)
   * 295156 - 1 (one mail on merge proposals)
   * 297640 - 1 (quote comments in web ui)
   * 297669 - 1 (Approve a mp via email)
   * 301836 - 1 (dashboard - difficult to figure out what to do)
 * If there's time, the old boring script
   * Next meeting
   * Action items
   * Queue status
   * Mentoring update
     * al-maisan looking for a mentor
   * Getting graphs in place before rolling out features (way before) - beuno
 * Action items
 * invite other teams to do lazr-js code reviews? [mars/barry]
 * Reminder: Who can do JS reviews? All reviewers? [henninge,allenap]
 * Proposed coding standard for YUI modules. [Edwin]
   * I think a file such as lib/canonical/launchpad/javascript/'''registry/timeline'''.js should define its module like this: {{{
YUI().add('registry.timeline', function (Y) {
    var module = Y.namespace('registry.timeline');
    module.someFunction = function() {};
});
}}}
   * Then, the code could be accessed with: {{{
YUI().use('registry.timeline', function(Y) {
    Y.registry.timeline.someFunction();
});
}}}
   * The [[JavaScriptReviewNotes#Namespaces|javascript style guide]] doesn't specify how modules should be named currently.
   * [[https://pastebin.canonical.com/25818/|Current inconsistencies]]
 * Cleaning up outstanding '''approved''' branches on [[https://code.edge.launchpad.net/launchpad/+activereviews| +activereviews]] [bac]
 * New developers as mentats? [bac]
 * Meeting frequency [bac]

 * Peanut gallery (anything not on the agenda)
Line 43: Line 49:
 * your name, date, reason  * ''yourname, date''
Line 47: Line 53:
 * barry to look into techniques for eliminating back-patching of schema types (avoiding circular imports)  * intellectronica and barry to draft guidelines for drive-by cleanups
Line 53: Line 59:
   * ''your item [your name]''
Line 56: Line 65:
 * [[ReviewerMeeting20100106]]
 * [[ReviewerMeeting20091125]]
 * [[ReviewerMeeting20091014]]
 * [[ReviewerMeeting20091007]]
 * [[ReviewerMeeting20090930]]
 * [[ReviewerMeeting20090923]]
 * [[ReviewerMeeting20090916]]
 * [[ReviewerMeeting20090909]]
 * [[ReviewerMeeting20090902]]
 * [[ReviewerMeeting20090826]]
 * [[ReviewerMeeting20090819]]
 * [[ReviewerMeeting20090812]]
 * [[ReviewerMeeting20090805]]
 * [[ReviewerMeeting20090729]]
 * [[ReviewerMeeting20090722]]
 * [[ReviewerMeeting20090715]]
 * [[ReviewerMeeting20090708]]
 * [[ReviewerMeeting20090701]]
 * [[ReviewerMeeting20090624]]
 * [[ReviewerMeeting20090617]]
 * [[ReviewerMeeting20090610]]
 * [[ReviewerMeeting20090603]]
 * [[ReviewerMeeting20090513]]
 * [[ReviewerMeeting20090506]]
 * [[ReviewerMeeting20090429]]
 * [[ReviewerMeeting20090422]]
 * [[ReviewerMeeting20090415]]
 * [[ReviewerMeeting20090408]]
 * [[ReviewerMeeting20090318]]
 * [[ReviewerMeeting20090311]]
 * [[ReviewerMeeting20090304]]
 * [[ReviewerMeeting20090225]]
 * [[ReviewerMeeting20090218]]
 * [[ReviewerMeeting20090211]]
 * [[ReviewerMeeting20090204]]
 * [[ReviewerMeeting20090128]]
 * [[ReviewerMeeting20090121]]
 * [[ReviewerMeeting20090114]]
 * [[ReviewerMeeting20090107]]
 * [[ReviewerMeeting20081217]]
 * [[ReviewerMeeting20081210]]
 * [[ReviewerMeeting20081209]]
 * [[ReviewerMeeting20081203]]

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 BradCrittenden 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 06-Jan-2010 1500 UTC - 1545 UTC

  • AsiaPac Wednesday 06-Jan-2010 2100 UTC - 2145 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
  • invite other teams to do lazr-js code reviews? [mars/barry]
  • Reminder: Who can do JS reviews? All reviewers? [henninge,allenap]
  • Proposed coding standard for YUI modules. [Edwin]
    • I think a file such as lib/canonical/launchpad/javascript/registry/timeline.js should define its module like this:

      YUI().add('registry.timeline', function (Y) {
          var module = Y.namespace('registry.timeline');
          module.someFunction = function() {};
      });
    • Then, the code could be accessed with:

      YUI().use('registry.timeline', function(Y) {
          Y.registry.timeline.someFunction();
      });
    • The javascript style guide doesn't specify how modules should be named currently.

    • Current inconsistencies

  • Cleaning up outstanding approved branches on +activereviews [bac]

  • New developers as mentats? [bac]
  • Meeting frequency [bac]
  • Peanut gallery (anything not on the agenda)

Apologies

Please include the date for which you will be unavailable.

  • yourname, date

Outstanding Actions

  • intellectronica and barry to draft guidelines for drive-by cleanups

Notes

Proposed items

  • your item [your name]

Rejected items

Summaries of previous meetings

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