Skip to content

Conversation

@romansk
Copy link

@romansk romansk commented Oct 1, 2013

No description provided.

@aterrien
Copy link
Owner

aterrien commented Oct 1, 2013

Thank you Roman,

What do you think to let jQuery do it ? It seems that jQuery supports AMD since 1.7 :

if ( typeof define === "function" && define.amd && define.amd.jQuery ) {
    define( "jquery", [], function () { return jQuery; } );
}

@fidothe
Copy link
Contributor

fidothe commented May 6, 2014

Unfortunately plugins need to support AMD as well. I've added a much cleaner version of this PR in #202, which only adds the UMD boilerplate and doesn't alter the rest of the code at all.

@aterrien
Copy link
Owner

aterrien commented May 6, 2014

#202 was merged, thanks @fidothe.

@aterrien aterrien closed this May 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants