Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added AMD support. #94

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
1 participant

samcday commented Nov 5, 2011

Hi,

This patch enabled AMD loading of CoffeeKup in the browser. I've tested these changes using RequireJS and standard script tag loading in the browser.

I set the dependency on CoffeeScript to use the tag "coffee-script", since the coffee-script browser distributable js file is named this way.

It's probably best not to land this patch until/unless jashkenas/coffee-script#1826 is merged. Either that or explain to people that they'll need to write a AMD shim for CoffeeScript in order for CoffeeKup AMD loading to work properly.

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment