[00:03] <veebers> elopio: ping, hey you available to do a MP review for me?
[00:05] <elopio> veebers: I am. Give me the link please.
=== thomi_ is now known as thomi
[00:09] <veebers> elopio: awesome, https://code.launchpad.net/~veebers/autopilot-qt/enable-datamodel-elements-add-new-nodes/+merge/224061
[00:11] <veebers> elopio: will you be around much longer? I was about to get lunch but can hold off if you want me around to query etc.
[00:12] <elopio> I'm going to have dinner soon but then I'll be back.
[00:12] <elopio> I can leave you comments on the MP and we can chat after eating.
[00:13] <elopio> veebers: one big details is that I have no idea what I'm reviewing here.
[00:13] <elopio> I've never seen autopilot-qt code before.
[00:13] <thomi> elopio: there's a first time for everything :)
[00:14] <veebers> elopio: ah right, well we can hangout quickly if you want and can give you a quick overview
[00:15] <veebers> elopio: thomi has (and I'll get a re-review too) reviewed so between the 2 of you it should get good coverage for AP related things as well as general code quality
[00:15] <elopio> veebers: I'll just read it and get some context first.
[00:15] <veebers> elopio: sweet, I'll head out for lunch. Thanks again :-)
[00:38] <elopio> veebers: I'm going out for dinner. I'll continue with the branch when I'm back.
=== rvr is now known as rvrOff