Aron Carroll aron

Organizations

@okfn @asyncjs @skiffhack @jsbin
@aron
Annotations showing on multiple pages
aron commented on issue aron/annotator.offline.js#3
@aron

I'm not sure what you want jQuery to be doing here. You need to pass it a CSS selector not a piece of text. If you want to get the title of the pag…

aron commented on pull request mroderick/bogus#6
@aron

Looks good to me, other than those commented out lines. Nice feature too. :boat:

aron commented on pull request mroderick/bogus#6
@aron

Same here...

aron commented on pull request mroderick/bogus#6
@aron

Remove this?

aron pushed to master at aron/dotfiles
@aron
  • @aron 48c4fa1
    Add iTerm preferences file
aron pushed to master at aron/dotfiles
@aron
aron closed pull request aron/nikeplus-to-runkeeper#4
@aron
fixes for activity time and calories
aron commented on pull request aron/nikeplus-to-runkeeper#4
@aron

Thanks, merged in 46cadb3

@aron
@aron
Lanyrd API proxy is down
@aron
aron commented on pull request asyncjs/asyncjs.github.com#77
@aron

@aaunel ah okay, thats great thanks for the information!

aron opened pull request asyncjs/asyncjs.github.com#81
@aron
Clean up the installation documentation
4 commits with 150 additions and 11 deletions
aron opened pull request asyncjs/asyncjs.github.com#80
@aron
Restores venue maps
2 commits with 13 additions and 1 deletion
aron opened pull request asyncjs/asyncjs.github.com#79
@aron
Include analytics only if deployed to production
1 commit with 2 additions and 1 deletion
aron commented on pull request asyncjs/asyncjs.github.com#77
@aron

Also small nitpick, but the path is at /feed/atom but the resource is now rss. I guess that's not a big deal if it keeps peoples readers up to date.

aron commented on pull request asyncjs/asyncjs.github.com#77
@aron

Hey, thanks for finally fixing this :smile: Looking at the code, other than showing 10 rather than 5 posts, I can't see any difference between the two lo…

aron commented on pull request asyncjs/asyncjs.github.com#78
aron commented on pull request asyncjs/asyncjs.github.com#78
@aron

@aaunel that site.pygments check was there to only enable the analytics when deployed to GitHub pages (i.e. in production), GitHub used to force set

aron commented on pull request hypothesis/h#1941
@aron

You're right self refers to the template. It's used to call the same block multiple times in a template. If you want to print a block multiple tim…

aron commented on issue hypothesis/h#1963
@aron

The method is defined inside the constructor and CoffeeScript does not support super calls in these cases. The error is being thrown because Annota…

aron commented on issue hypothesis/h#1956
@aron

@dwhly I'd assume so too. @BigBlueHat as mentioned in #1954 I believe it's failing due to the add-on embedding an older version of h into the docum…

aron commented on issue hypothesis/h#1954
@aron

Hey, so I forgot to document this. The Firefox extension is currently built from this PR #1813 as we require unminified assets. It's pretty outdate…

aron commented on pull request hypothesis/h#1941
@aron

Aha, so would then it be better to import horus.lib and call it via horuslib.FlashMessage or something...

aron commented on pull request hypothesis/h#1941
@aron

Could not for the life of me get this patch to work. I've no idea what I'm doing wrong, but the claim_account method runs with the original FlashMe…