= ReviewerMeeting20090304 = === summary === * all OCR reviewers are encouraged to review javascript * al-maisan graduates * noodles joins us as a javascript reviewer now, full mentat next cycle === log === {{{ Mar 04 10:00:43 #startmeeting Mar 04 10:00:44 Meeting started at 09:00. The chair is barry. Mar 04 10:00:44 Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE] Mar 04 10:00:56 hello everyone and welcome to this week's ameu reviewers meeting. who's here today? Mar 04 10:00:57 me Mar 04 10:01:02 me Mar 04 10:01:03 me (at sprint - so half here) Mar 04 10:01:03 me Mar 04 10:01:06 me Mar 04 10:01:09 and me Mar 04 10:01:40 me Mar 04 10:01:41 me Mar 04 10:01:42 welcome noodles775 ! Mar 04 10:01:49 * noodles775 grins Mar 04 10:02:05 me Mar 04 10:02:39 bac: ping Mar 04 10:02:47 me Mar 04 10:02:59 danilos: ping Mar 04 10:03:03 EdwinGrubbs: ping Mar 04 10:03:16 mars: ping Mar 04 10:03:25 rockstar: ping Mar 04 10:03:34 salgado: ping Mar 04 10:03:44 me Mar 04 10:03:46 [TOPIC] agenda Mar 04 10:03:47 New Topic: agenda Mar 04 10:03:58 pretty light day today! Mar 04 10:04:00 * Roll call Mar 04 10:04:00 * Peanut gallery (anything not on the agenda) Mar 04 10:04:00 * Mentoring update Mar 04 10:04:00 * al-maisan graduates Mar 04 10:04:00 * noodles to start js reviews this cycle, official mentat status next cycle Mar 04 10:04:00 * Action items Mar 04 10:04:21 [TOPIC] peanut gallery Mar 04 10:04:22 New Topic: peanut gallery Mar 04 10:04:33 me Mar 04 10:04:41 let's start off with this: does anyone have anything reviewish that they'd like to bring up? Mar 04 10:05:58 wow, nothing? Mar 04 10:06:23 js reviews Mar 04 10:06:39 can be difficult to organise when all the reviewers are sprinting Mar 04 10:07:10 mars: do we have a plan for mentoring more js reviewers? Mar 04 10:07:19 Do we want to consider a plan to train up js reviewers? Mar 04 10:07:32 me Mar 04 10:07:33 barry, anyone can pick up a JS review, if they are comfortable doing so Mar 04 10:07:34 ideally everyone should be able to review js Mar 04 10:08:12 sure, i'm just wondering if we need a mentoring program for those (like me) who feel less comfortable reviewing js. Mar 04 10:08:14 frankly, I think i need much more experience writing JS code before I would feel able to review JS code Mar 04 10:08:18 sinzui can't review his own JS code, but he can ask the OCR if they are comfortable doing so Mar 04 10:08:24 :) Mar 04 10:08:50 hmm Mar 04 10:09:36 the OCR/reviewer thing is the best we have for now - it spreads the knowledge pretty wide Mar 04 10:09:56 without having formal JS-Review training for all the other reviewers Mar 04 10:10:16 mars: i guess we just need backups then for stuff we're less experienced with? Mar 04 10:10:32 mars: Do we have updated standards and cheatsheet of common JS problems? Mar 04 10:10:44 barry, well, we do have 8 people who can review JS code Mar 04 10:10:55 sinzui, that's on my list to write today Mar 04 10:11:09 * sinzui is thinking of the rule that zope3 does not make css3 valid ids, so Y.get() fails. Mar 04 10:11:20 barry: in a way, reviewing is a great way to build confidence, because you _have_ to understand what the code does and why it's written the way it is, and you _should_ ask if you don't :) Mar 04 10:11:20 mars: i actually like the idea of throwing people into the deep end, getting the ocr to review the js, but with an experienced js'er reviewing the review Mar 04 10:11:25 sinzui, :) Mar 04 10:11:48 intellectronica: yep :) Mar 04 10:12:14 barry, if you were at the sprint, you may have rethought that... Mar 04 10:12:22 lol Mar 04 10:12:34 the deep end for browser scripting is... deep Mar 04 10:12:36 with sharks Mar 04 10:12:44 :) Mar 04 10:13:00 mars: foundations will make it all better, right? Mar 04 10:13:05 that's why I feel it's best suited to a "volunteer" program Mar 04 10:13:19 mars: or are you fitting laser helmets right now? Mar 04 10:13:23 barry, we're working on it - tools should help Mar 04 10:13:52 barry, yes, tools will help Mar 04 10:14:16 in a few cycles we should be to the point where the reviewer can just trust the tests Mar 04 10:14:37 but we're not there yet Mar 04 10:14:40 mars: i think coding guidelines will help a lot, even if they're incomplete Mar 04 10:14:49 mars: something a reviewer can read and point to Mar 04 10:14:51 agreed Mar 04 10:14:58 mars: that we fill up as questions arise Mar 04 10:15:21 mars: so i guess the recommendation is: ocr should try to review, and ask if you're not sure Mar 04 10:15:37 barry, true - the JS review then becomes more of a style and conformance review, but it still has value Mar 04 10:15:47 also agreed Mar 04 10:15:59 mars: i think that's the best way to start, since those are easier to get a handle on Mar 04 10:16:09 oh, i also want to point out that mars updated https://dev.launchpad.net/ReviewerSchedule Mar 04 10:16:25 this now includes a "specialties" column so you have a better idea of who can help out with js Mar 04 10:16:50 sinzui: thanks. did you have anything else on this topic? Mar 04 10:16:56 no Mar 04 10:17:17 cool, anybody have anything else on any topic not on the agenda? Mar 04 10:17:18 s/specialties/mentoring offered/ ? Mar 04 10:17:30 mars: sup to the reviewer i guess Mar 04 10:18:27 [TOPIC] mentoring update Mar 04 10:18:28 New Topic: mentoring update Mar 04 10:18:35 * al-maisan graduates Mar 04 10:18:43 but he's not here :) Mar 04 10:18:58 congart al-maisan! Mar 04 10:19:04 * noodles to start js reviews this cycle, official mentat status next cycle Mar 04 10:19:30 noodles775: we'll figure out a mentor/ocr for you next cycle. welcome aboard! Mar 04 10:19:36 Thanks! Mar 04 10:20:10 [TOPIC] action items Mar 04 10:20:16 New Topic: action items Mar 04 10:20:19 * gary to add `getStore()` as an alias for `_get_store()` Mar 04 10:20:32 gary's not here though. does anybody know if this happened? Mar 04 10:21:15 * flacoste to look into storm/sqlobject result set compatibility Mar 04 10:21:34 barry: gary still needs to do this, he's on a sprint so has an excuse Mar 04 10:21:42 cool :) Mar 04 10:21:47 barry: i replied with a possible solution to gavin yesterday Mar 04 10:21:54 barry: he can tell if it works or not :-) Mar 04 10:22:01 excellent, thanks! Mar 04 10:22:05 flacoste: I'll be putting that in my branch later today. Mar 04 10:22:12 Thank you :) Mar 04 10:22:16 allenap: great, let us know how it works Mar 04 10:22:23 allenap: if it works, i'd suggest you add it to storm itself Mar 04 10:22:24 Will do. Mar 04 10:22:31 well, *if* it works :) Mar 04 10:22:38 flacoste: Good idea! Mar 04 10:23:01 * gary will check to see if there's a bug open for adding a hook to `bzr send`, and submit one if there isn't Mar 04 10:23:09 gary's sprinting so we won't worry about this one Mar 04 10:23:16 * bigjools to take crack at helper functions for backpatching schemas to avoid circular imports Mar 04 10:23:23 and bigjools isn't here (sprinting?) Mar 04 10:23:31 * abentley to experiment on storm base class Mar 04 10:23:50 nor is abentley, but does anybody know what the status of his sugarstorm branch is? Mar 04 10:24:55 i have not done my three action items, but now i have access to to the old wiki again, so i'm going to try to find some time to migrate/update Mar 04 10:25:03 * flacoste to work on API reviewer cheat sheet Mar 04 10:25:07 * flacoste hides Mar 04 10:25:31 no worries hoover :) keep it on the list? Mar 04 10:26:05 barry: bigjools is off (vacations) this week Mar 04 10:26:16 cprov: cool, thanks Mar 04 10:26:33 well. that's everything that i have. if there's nothing more we can break early Mar 04 10:26:48 5 Mar 04 10:27:10 4 Mar 04 10:27:20 2 Mar 04 10:27:30 1 Mar 04 10:27:40 #endmeeting }}}