/srv/irclogs.ubuntu.com/2013/06/04/#juju-gui.txt

hatchhuwshimi: ok done00:06
hatch;)00:06
gary_posterhatch uistage isn't broken.  zoom out.  silly kids00:06
hatchhuwshimi: http://jsbin.com/itaqem/2/edit click the box :)00:07
hatchgary_poster: lol Uncaught TypeError: Cannot read property 'browser_enabled' of undefined  is what I see00:07
gary_posterhatch, wfm.  hard reload?00:09
huwshimihatch: Nice! Now you just need to make it close when you click outside of it, make it work with the keyboard and build a YUI widget to handle the creation etc ;)00:10
gary_posterhttp://uistage.jujucharms.com:8080/ and http://uistage.jujucharms.com:8080/:flags:/browser_enabled/00:10
hatchhuwshimi: umm....yeah...about that.... :P00:10
huwshimihehe00:10
hatchgary_poster: ohh here is what happened Application Cache Error event: Failed to parse manifest http://uistage.jujucharms.com:8080/juju-ui/assets/manifest.appcache00:10
hatchwe didn't properly set it to clear the appcache before we updated it00:11
hatchso the index.html is still cached on my system00:11
gary_posterhatch, ah.  so yeah, clear with chrome://appcache-internals/ and prob fine00:11
hatchthis brings up something curious to me....what is the proper way to remove appcache on a live site00:12
hatchhmm00:12
hatchok gota run, will be back later to try and make 'click outside' to work with css :P00:14
hatch^ was a joke :)00:15
gary_poster:-)00:15
huwshimi:)00:18
=== rogpeppe2 is now known as rogpeppe
gary_posterhey bac, do the recent jenkins failures look like they have anything to do with images?12:28
gary_posterdoesn't look like it to me :-/12:29
baclooking12:29
gary_posterlooks like random stupidity and a reminder that we should switch to a more production-centric system12:30
bacgary_poster: yeah, it looks like a bunch of hooha to me.  CI should not require so much finger crossing and chicken sacrificing.12:35
gary_posteryeah :-(12:35
bacjujugui: so yesterday afternoon my ISP dudes couldn't find parking so they just left.  now they are circling the block again.  connectivity may (or may not) be flaky soon.12:36
gary_posterI guess the only things stopping us moving to ec2 are (1) corporate account, or me taking one for the team; and (2) opening up the firewall from the test center to ec2.12:37
gary_posterThe first one is the one that slows me down the most. :-/12:37
bacgary_poster: heck, i'll volunteer my AMEX.  think of the points!12:37
bacno, really12:38
gary_poster:-)12:38
gary_posterbut then anyone who can log into the machine can get your ec2 creds12:38
benjibac: that's hilarious12:38
bacoh12:40
bacso, carlos is here to replace my radio.12:40
hazmatbac, wow.. thats silly.12:40
bacer?12:40
hazmatbac, isp dudes leaving, cause they can't find parking..12:40
bacoh, yeah, well...12:40
hazmatafternoon cerveza sounds more apt12:41
baci would've gladly given them a cerveza12:41
benjigary_poster: I'm in guichat13:01
gary_posterbenji, darn, ok13:02
gary_postercoming13:02
* bac o/13:16
hatchmorning13:53
hatchlooks like canonistack errors on jenkins13:55
bachi abentley, do you have a moment to answer some bzr lightweight checkout setup questions?14:02
abentleybac: Sure.14:02
jcsackettorangesquad: should we move the green categories card into the coding lane? I think it is now bound on rick's card as the last thing that needs to land for it to be done, yeah?14:04
jcsackettor does it just sit and go to done when we're done with other cards?14:04
rick_h_jcsackett: seems strange to have a green card in coding without a face/etc on it14:04
jcsackettrick_h_: don't disagree, but having it in "Next" when all activity on it is going seems weird too.14:05
jcsackettrick_h_: : in the case of a review with lbox -req (i.e. a prereq branch) does lbox submit properly merge the branch, or does it just apply the patch set?14:11
rick_h_jcsackett: oh hmm, I think since it's a req the other one needs to land first? It just builds the right diff/MP14:11
rick_h_jcsackett: I don't think you can submit both in one swoop. 14:12
rick_h_jcsackett: but I could be wrong, not tried it tbh14:12
jcsackettrick_h_: yeah, i don't really think now's the time to experiment, since it'll break the build if it doesn't work. :-P14:16
gary_posterbenji, gustavo suggested that we use full ip address, not merely port.  Could you take a look at the only non-resolved comment on https://docs.google.com/a/canonical.com/document/d/14W14YAPYdivL9nF7Wvrb6kHX0xrsxV0fAEMfXNt7L3Q/edit# and see if you can see where to incorporate the idea, or the question (since I don't really understand where he is coming from) in the new doc?14:28
benjigary_poster: I can't claim to understand the comment containing the explanation.14:30
gary_posterbenji, you and me both then.  Maybe add a comment in the new doc highlighting the fact that we still need to resolve this with Gustavo?14:31
sinzuiorangesquad. I was/am leaving the feature cards in the next lane because they remind us that the task/defect/improvement cards should relate to the features. I move the features to acceptance when I think we have delivered all the work. This is a bit risky since acceptance can find issues that will go into the next lane14:35
teknico1bac: y u no put python-charmhelpers in juju-gui-charmers ppa? charm deploy sad14:36
sinzuiI don't think the story1 and story2 makes sense if we continue to let the feature cards define the theme of work14:36
teknico1BradCrittenden: ^^14:37
hatchgary_poster: so I'm reading through this doc from luca__ and I'm wondering how I give feedback, is there some way to comment on these points?14:39
luca__hatch: which parts do you want to comment on?14:40
BradCrittendenteknico1: sorry, the charm-tools in stable is from juju/pkgs and does not include the python-charmhelpers.  i am currently copying it from juju-gui-charmers/devel14:40
=== BradCrittenden is now known as bac
hatchluca__: so far Route A and Route B - I feel that we should have some kind of 'setting up' indicator to indicate that it's being set up instead of just hanging around as a ghost for 15 minutes14:40
luca__hatch: you can add that to the comments section at the bottom of the post14:41
luca__hatch: if you click into it14:41
hatchahh ok14:41
* hatch votes to use google docs next time :P14:41
bacteknico1: unfortunately it'll take at least an hour14:41
hatchluca__: also """Is there ever a case where you just want to add charms without reading the details?""" yes I do with npm all the time14:42
luca__hatch: https://sites.google.com/a/canonical.com/juju-meeting-notes/scenarios-1/buildinganenvironment14:42
teknico1bac: the charm tries to install the "python-charmhelpers" package, it's not going to be happy if it does not find it in the ppa14:42
luca__hatch: great :)14:42
teknico1bac: or am I missing something?14:42
hatchluca__: :) I coudln't comment on that section so I'll leave that to you14:43
bacteknico1: i understand.  that's why i just told you my actions to rectify the problem.14:43
luca__hatch: I've just changed the user needs to a separate post so it can be commented on14:43
hatchoh ok awesome14:43
hatchI've never heard of sites.google.com before14:43
teknico1bac: then it's me the one who's not understanding :-) what do you mean by "the charm-tools [...] does not include the python-charmhelpers"?14:44
bacteknico1: by "i am currently copying it" i mean on launchpad i have requested the package be copied from the devel PPA to the stable PPA14:44
teknico1bac: but I don't see it in the devel PPA either...14:45
teknico1that is, here: https://launchpad.net/~juju-gui-charmers/+archive/devel14:45
bacteknico1: it's weird.  you have to click on 'details' and then look at charm tools.  it'll show you the packages withing that, uh, package14:45
bacthe charm-tools source package builds three different installation packages.  python-charmhelpers is one of them14:45
baclast week the build recipe was changed to *not* build python-charmhelpers for ppa:juju/pkgs14:46
teknico1bac: oh my gosh, I see it now, thank you. way to go hiding stuff :-)14:46
bacteknico1: that was quite a mystery to unravel last week14:46
bacteknico1: anyway, i screwed up i getting the charm-tools from the wrong spot for stable14:47
teknico1bac: I wonder why that happened... (not building python-charmhelpers)14:47
bacteknico1: marco changed his debian/control file to explicitly delete it.  unsure why.14:47
bacteknico1: if you need to deploy a charm right now  you can set repository-location=ppa:juju-gui-charmers/devel14:48
bacotherwise the copy from one ppa to stable will complete in an hour or so14:48
teknico1bac: ok, thanks, at least I know what the problem is :-)14:48
* bac regrets having lost his PPA administration super powers14:49
* bac wonders if that is the cause of jenkins failures14:49
teknico1it would seem likely14:49
teknico1(bac: did you ask around? someone may have found them ;-) )14:49
gary_posterbac, maybe follow up with marco about python-charmhelpers?  does this mean that he broke gui deployments?14:53
bacgary_poster: i talked to him last week but decided to use my own build recipe for our devel and stable ppa.  while his recipe is no longer building p-ch it was not deleted from the juju/pkgs repo, though it is not visible via the UI.14:55
gary_posterok thanks bac14:55
gary_posterbac, maybe clarify with him that deleting it Would Be Bad?  Or is it not possible to delete?14:56
baci erred by testing against our new devel ppa and not stable14:56
gary_posterIf it is possible to delete, maybe that would be an answer to our "what do you do if you have a package in devel that doesn't work out" question from yesterday?14:56
bacgary_poster: it is not possible to delete unless he were to delete the entire charm-tools source package from that ppa, which he has no reason to do14:56
gary_posterok14:56
gary_posterthanks14:56
bacgary_poster: what is the risk you see now that our charm does not rely on juju/pkgs anymore?14:57
gary_posterbac, old charms only.  but until you push the new charm with dependencies on the new ppa to ~juju-gui-charmers (did you do that?), the main charm is the old charm14:58
bacgary_poster: that was done:  https://code.launchpad.net/~juju-gui-charmers/charms/precise/juju-gui/trunk15:00
gary_postercool bac, thanks.15:00
gary_posterbac, then I guess I'm not too worried.  Paranoid, but not worried. :-P15:00
bacbut i think it was premature and not following our guidelines15:00
gary_posterbecause it was released prior to full testing?15:00
bacit should gone to the old spot but due to my broken setup i pushed to lp:charms/juju-gui.  (so my new blog post is wrong)15:01
gary_posterah ok15:01
bacyes, the official charm is currently broken until the PPA publishing for stable is complete15:01
gary_poster:-/ k15:02
bacsoory15:02
gary_posteryou're on it15:02
gary_posteryou are fixing it15:02
gary_posterI suppose we should have another "what can we do so this doesn't happen again" conversation, in which we all say "use tarmac"15:03
teknico1bac: while you're at it, please also fix my head and my wall ;-)15:03
gary_posterwant to ask matsubara where we are on that but have not seen him15:04
gary_posterhey luca__ , could you copy over the things you want us to comment on to google doc instead?15:05
gary_posterI'd like to use the google doc line by line thing15:05
luca__gary_poster: sure, I'll do that once I get out of this meeting15:05
gary_posterthank you!15:05
bacjujugui: to avoid the charm problem i just had, everyone should verify lbox is merging to lp:~juju-gui/charms/precise/juju-gui/trunk not lp:charms/juju-gui15:07
gary_posterbac, I suggest we make a .lbox file in charm , like the one in gui15:07
gary_posterinstead of 15:08
bacthis blog post has bzr setup instructions if using lightweight checkouts: http://jujugui.wordpress.com/2013/06/04/bazaar-lightweight-checkouts-and-charm-development/15:08
gary_posterpropose -cr -for lp:juju-gui15:08
gary_posterwe would simply have15:08
gary_posterpropose -cr -for lp:~juju-gui/charms/precise/juju-gui/trun15:08
gary_posterk15:08
gary_posterand then anyone checking out the branch would have the proper lbox behavior immediately15:08
gary_posterAIUI at least15:08
bacgary_poster: ok, i'll add that now.  w15:14
gary_posterthanks bac15:14
alejandraobregongary_poster: luca and i just had the meeting with charline15:16
gary_posteralejandraobregon, cool!15:17
gary_postercall?15:17
alejandraobregonwrote up comments and suggestions beneath your current questions15:17
alejandraobregonhttps://docs.google.com/a/canonical.com/document/d/1bSR1nfSsZs4XQoTv1IZu-so5FM-_R4JsvRsW3IAZyag/edit15:17
hatchbcsaller: are you in yet?15:17
gary_posterlooking15:17
luca__hatch: I've added google docs to the posts15:21
luca__gary_poster: ^15:21
hatch:) You didn't have to15:21
hatchbut thanks!15:21
hatch:)15:21
hatchI'll move my comments there15:22
bcsallerhatch: I am now15:22
hatchI am writing the viewlet too if that's alright - I can't really do much without it15:22
gary_posteralejandraobregon, really fantastic, big thanks to you, Luca, and Charline.  I'll make a few small revisions, and then send it to the lawyer.15:23
* gary_poster needs to step away15:23
gary_posterback in a few15:23
bcsallerhatch: sounds good15:23
alejandraobregongary_poster: no wories15:23
alejandraobregonworries15:23
hatchbcsaller: https://gist.github.com/hatched/2463d15dd51d12e04258 got a sec for a guichat?15:27
hatchgist updated15:40
gary_postercongrats on landing core branch, frankban :-)15:45
bcsallerhatch: I can hop in chat now15:46
frankbangary_poster: \o/15:46
gary_poster:-)15:46
hatchbcsaller: ok there15:47
rick_h_luca__: ping, any chance we can get these category icons in 160x160 for the full charm details icon view please? https://drive.google.com/a/canonical.com/#folders/0B1IM--9A1RkTMndkVG8zeHFlVmM15:51
luca__rick_h_: I have no idea, let me ask the guy who made them :P15:52
rick_h_luca__: thanks, appreciate it15:52
rick_h_luca__: I had three sizes and ran with it and now see that we changed the sizes so I've got 2/3 I need to complete the category work15:52
luca__rick_h_: He's going to make the new sizes, hopefully won't take too long. I'll add them to the drive and email you when you they are up.15:55
rick_h_luca__: appreciate it15:55
gary_posterjujugui kanban now call in 415:56
gary_posterjujugui call in 115:59
abentleyorangesquad: Could you please review https://code.launchpad.net/~abentley/charmworld/remove-api-0/+merge/167324 ?16:00
* sinzui does16:00
luca__rick_h_: I've just uploaded the 160x160px icons to that folder: https://drive.google.com/a/canonical.com/#folders/0B1IM--9A1RkTMndkVG8zeHFlVmM16:04
sinzuiabentley, is test_reindexed_no_client_charms testing that an error is not raised? There is no asserts.16:05
abentleysinzui: Yes.16:06
abentleysinzui: Should I add asserts or a comment?16:06
sinzuiA comment would be helpful16:07
abentleysinzui: I've added a similar comment to test_create_replacement_misssing16:09
sinzuithank you abentley. r=me.16:10
abentleysinzui: Thanks.16:10
hatchbcsaller: reasonable to assume that we could have something like https://gist.github.com/hatched/3876ff1fc8ca1bff5bb0 for the view-container?16:17
julianwajcastro: ping16:17
jcastrojulianwa: pong!16:19
julianwajcastro: hi, I had a problem to login juju-gui16:21
julianwajcastro: juju gui is deployed by charms in charm store, for python juju stable version16:21
* gary_poster listens16:22
julianwabut failed to login with admin-secret key16:22
julianwa2013-06-04 11:56:22,399: juju.rapi.ws@INFO: Invoking method <bound method APIContext.login of <juju.rapi.context.APIContext object at 0x22940d0>> with {u'password': u'FDU2K9N7bsJZkfh6s6:829z85ytjayxqYKLPU:Tvmtbwcku9FsSyjuVWzPxmAEBUaBjA9E', u'user': u'admin'}16:22
julianwa2013-06-04 11:56:24,092: juju.rapi.ws@INFO: Returning results {'failure': 'Traceback (most recent call last):\nFailure: zookeeper.NoAuthException: not authenticated\n', 'request_id': 1, u'user': u'admin', 'err': True, 'op': u'login', u'password': u'FDU2K9N7bsJZkfh6s6:829z85ytjayxqYKLPU:Tvmtbwcku9FsSyjuVWzPxmAEBUaBjA9E', 'log': [('error', 'Invalid credentials')]}16:22
bcsallerhatch: wouldn't it be a loop around viewlets where you put the container class use for switching around them automatically?16:23
julianwajcastro  gary_poster, any ideas?16:24
gary_posterjulianwa, do you have multiple environments in your ~/.juju/environments.yaml file?  We haven't encountered any problems like that ourselves, so my first guess woudl be that you used the wrong password for the active environment16:24
gary_posterback in a sec16:25
julianwagary_poster: thanks, I will check :)16:25
BradCrittendenjujugui: would like one review of the code backport just for sanity check: https://codereview.appspot.com/9975046/16:28
=== BradCrittenden is now known as bac
hatchbcsaller: updated gist https://gist.github.com/hatched/3876ff1fc8ca1bff5bb016:28
hatchthe specifics we can work out after design - but the concept should be similar16:29
bacjujugui: python-charmhelpers is now properly in stable PPA: https://launchpad.net/~juju-gui-charmers/+archive/stable/+build/464111416:35
gary_posterthank you bac16:35
teknico1bac: thank you16:35
bacgary_poster, teknico1: can either of you do the quick review i requested?  i'd like to get this landed to finish cleaning up the mess.16:37
gary_posterI will if teknico1 can't.  Otherwise trying to finish something up.  teknico1, are you already at EoD?16:38
teknico1gary_poster: I'll squeeze it in, it seems simple enough16:40
gary_posterthanks teknico1 16:40
teknico1bac: it looks exactly the same as yesterday's branch, isn't it?16:41
bac:)16:42
baci did say sanity check16:42
bacthat's the nature of a backport16:42
teknico1bac: done16:42
bacthanks16:42
=== deryck is now known as deryck[lunch]
rick_h_jcsackett: sinzui can I get some review <3 please? https://codereview.appspot.com/9975047 note that it's blocked on kanban until we get the 160px category images but that doesn't effect the code at all. 16:52
rick_h_hatch: ^^ as well if you have a chance16:52
hatchalright16:55
rick_h_hatch: ignore all the svg files :P16:56
rick_h_#DiffsAreSmallerThanTheyAppearInMirror16:56
hatchlol16:57
Makyogary_poster, bcsaller - would like to start "zoom to a service if any on first delta".  It sounds reasonable, but I'm game for a pre-imp call if either of you want.17:04
gary_posterMakyo, don't need one from me :-)17:04
bcsallerMakyo: not unless you want one17:04
gary_posterbac, you done with your critical card in Maintenance?17:04
Makyogary_poster, bcsaller - Alright.  Will start for now.17:05
rick_h_sinzui: ping, got a sec?17:05
hatchrick_h_: review done17:07
rick_h_hatch: cool, I'm changing it up right now :)17:08
bacgary_poster: moved17:09
rick_h_hatch: chat?17:09
gary_posterthanks bac17:09
hatchsure17:09
sinzuirick_h_, I do. Sorry for the delay. I had a late lunch to argue with my ISP17:21
rick_h_sinzui: invite inbound17:24
hatchI am having an issue generating a new template file17:28
hatchno matter what I do it doesnt' add the new .handlebars file into the rollup17:28
hatchI tried make clean and then make again17:28
hatchbut no luck17:28
rick_h_hatch: so when you run that make command I give you do you get any error output?17:29
rick_h_hatch: if there's a syntax error in the template you'll get a one line in the make comment output that tells you 'something went wrong' and is partially helpful17:30
hatchi just rm'd the whole dir's and then remade17:30
rick_h_lol17:30
rick_h_well, copy the output next time you  make build-shared/juju-ui/templates.js 17:30
rick_h_I'll bet you have a mismatched tag or missing # or something17:30
* rick_h_ did that more than a few times the last couple of days17:31
hatchrick_h_: yup it was a syntax error17:31
hatch:/17:31
hatchargz17:31
rick_h_hatch: party on17:32
benjiok, gary_poster, I'm out of time (plus some).  Do you want to review the draft together or just make changes and annonce?17:36
gary_posterbenji, I need to actually run and do something else.  Let's have lunch and then reconvene?  I'm not too worried about it since I kind of announced it already via the calendar announcement.17:36
gary_posterdoes that sound ok to you?17:37
benjik17:37
gary_postercool, thank you17:37
jcsackettrick_h_: looking now.17:46
=== deryck[lunch] is now known as deryck
jcsackettrick_h_: nm, just saw curtis's comments hit.17:50
rick_h_jcsackett: yea thanks17:59
hatchgoing to grab some lunch17:59
abentleysinzui: It turns out my migration was bogus (it only updated a single document, and my tests only tested a single document).  Should I do a new migration or bugfix the existing one?18:04
sinzuiabentley, I think a new one.18:04
sinzuiabentley, Wouldn't a bug fix be more risky for a production deployment?18:05
abentleysinzui: We'd have to manually futz with staging to force migration 5 to run again.18:06
sinzuiwell, that is what I hoped a new migration would avoid.18:06
sinzuiabentley, how is a bug fix different?18:07
abentleysinzui: I was describing a bugfix.  A bug fix would update migration 5.  If we introduce migration 6, we still have to have migration 5, though I can change it to a no-op.18:08
sinzuiI like a new migration. making migration 5 a no-op is fine since we reserve the right to choose which migrations are no longer supported18:09
abentleysinzui: https://code.launchpad.net/~abentley/charmworld/really-remove-icon/+merge/16736418:25
* sinzui looks18:25
sinzuiah, like the RE g option when doing substitutions. r=me abentley18:28
abentleysinzui: Thanks.18:28
abentleysinzui: I would like it if we could remove migrations that are no longer supported, but the current code requires an unbroken numerical sequence.18:33
sinzuiabentley, I think we can live with dummies that just document the historical intent.18:34
rick_h__jcsackett: ping, did you want the background on the bar issue stuff at all or are you ok?18:40
hatchbcsaller: around?18:45
bcsallerhatch: yes18:45
hatchhave a moment for a screenshare?18:45
abentleysinzui: tarmac appears to have gone on strike, and I can't find matsubara.18:46
sinzuiabentley, I see18:48
=== _mup___ is now known as _mup_
sinzuiabentley, the only option I can think of at this time is to test and merge ourselves, then call upgrade charm18:51
abentleysinzui: I'm not rushed with this one.18:52
* benji_ cleans hundreds of messages about PyPI projects out of his inbox.18:57
* benji_ wonders when he grew a tail.18:57
=== benji_ is now known as benji
* benji feels much better now.18:58
=== _mup___ is now known as _mup_
hatchnew yui bugfix release http://www.yuiblog.com/blog/2013/06/04/yui-3-10-2-released19:40
hatchwe should look into upgrading to 3.10 at some point :)19:40
hatchjcastro: Great idea on the app infrastructure :)19:42
bacgary_poster: i've learned more about google analytics and it doesn't look like it is going to work for us.  they expect a single tracking code to be tied to a specific web site, which is not our use case.19:44
gary_posterbac, oh :-(19:44
bacwe want to track the use of an app that'll appear in lots of places19:44
gary_postery19:45
bacgary_poster: for instance, they won't even begin gathering data until their robot finds the tracking code at the specficied url ... which cannot include a custom port, btw19:45
bacso i cannot even test it on uistage19:45
gary_posterbac, any other options you found for our use case?19:46
bacnot yet19:46
baccrazy egg has the same setup19:48
gary_posterok :-/19:48
gary_posterI see for-fee ones that would seem to work19:49
gary_posteror maybe we have to set up our own app for this?  in which case this is a stretch goal19:52
gary_posterthat would be a shame19:52
gary_posterbah, benji, neither of us pinged the other and I've been doing other things.  looking at your doc19:54
benjik19:55
hatchgary_poster: you use sublime text 3 right? Are there any reasons to upgrade?19:59
gary_posterhatch, no, for me currently only anti-reasons.  hopefully better later19:59
hatchalrighty :)20:00
hatchI use this as my icon http://1024jp.deviantart.com/art/Sublime-Text-Icon-37383734720:01
hatchthought you might like20:01
gary_poster:-) cool 20:17
bacgary_poster: it *may* be possible to use ga if we get it bootstrapped with a fixed location20:19
gary_posterbac, how would that work?20:19
bacgary_poster: it looks like a code that their robots have verified at a fixed location can be used from other sites if you turn off the url filtering20:20
gary_posteroh20:20
gary_posterwell, we have jujucharms.com..20:20
bacyeah, that's true.  we also have uistage.jujucharms.com if we were on port 8020:21
gary_posterwe have that bac20:21
gary_posterhttp://uistage.jujucharms.com/20:21
bacoh, we do don't we?20:22
baci've been going to 8080 out of habit and bookmarks then20:22
gary_posterme too20:22
bacbenji: nm20:22
benjiI'm about to introduce a Y3K bug into the GUI.  I've made a note to myself to fix it in 2999.20:51
hatchyou better!20:53
gary_posterbenji, +1 on vitals proposal20:56
gary_postermade some changes20:56
benjicool20:56
gary_posterbenji, please review my last two comments/changes?20:56
* benji looks20:56
gary_posterBradCrittenden, are you optimistic or pessimistic about GA working out for us at this point?21:33
alejandraobregongary_poster: hey21:42
hatchtypeof null === "object" just bit me ....ouch it stings21:44
gary_posteralejandraobregon, hey21:56
hatchTDD works pretty damn well once you get the hang of it22:12
hatchhas anyone ever had any issues with setStyle on elements in mocha?22:33
hatchfor some reason it's throwing on a setStyle claming the method doesn't exist when it clearly does (from stepping through and manually calling it)22:34
hatchhmm it thinks nothing exists...22:37
hatchblah22:41
hatchfigured it out - it was reporting the wrong line :/22:50
bcsallerusing a source map and pretty print at the same time does break the line numbers22:51
hatchnow to figure out why THIS line doesn't work22:52
hatchheh22:52
hatchwell I'm baffled22:57
hatchwell I really wish I could solve this so I could commit this code and be done22:59
bcsallerI know the feeling23:04
hatchwell I'm not getting anywhere it's pushed up lp23:07
hatchhave a good night23:08

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!