Closed elmerbulthuis closed 11 years ago
+1 for merging this. Works fine with requirejs
I'm a big fat -1 against this as AMD breaks more than it solves. This is also the reason why we removed it a couple of days ago from Socket.io. AMD support should be solved and done by build systems not by libraries. Also: http://lisperator.net/blog/thoughts-on-commonjs-requirejs/
I'm -1 on it too, for the same reasons @3rd-Eden mentioned.
support for amd (requirejs and such) so the library can easily be used and optimized in a browser or server evironment via an amd enabled loader.