don't defer module definition unless using CommonJS-wrapped module #149

Closed
unscriptable opened this Issue Nov 13, 2012 · 1 comment

Comments

Projects
None yet
1 participant
Owner

unscriptable commented Nov 13, 2012

Right now, curl uses deferred dependency resolution whenever the module uses the "exports" or "module" pseudo-modules. It should only do this if the dev is also using the CommonJS-wrapped define() signature, as well:

define(function(require, exports, module) {});

It should not do this when the dev specifies the CommonJS pseudo-modules like this:

define(["require", "exports", "module", function(require, exports, module) {});

cc @bryanforbes

unscriptable was assigned Nov 13, 2012

Owner

unscriptable commented Apr 10, 2013

closing this since it's fixed in dev08 branch.

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