This repository has been archived by the owner on Jan 31, 2018. It is now read-only.

Support Simplified CommonJS Wrappers #33

Closed

stasm wants to merge 1 commit into mozilla:master from stasm:simplified-commonjs-wrapper

Contributor

stasm commented Jun 12, 2013

Add explicit support for CJS Wrappers which look like the following:

define(function(require, exports, module){
    var foo = require('foo'),
        bar = require('foo/bar');

    exports.doSomething = function(){
        console.log(foo + bar);
    };
});

A module is only treated as a CJS module if it doesn't contain a dependency array and the definition function contains at least one parameter.

More information:


          Support Simplified CommonJS Wrappers

a0cdb59

Add explicit support for CJS Wrappers which look like the following:

    define(function(require, exports, module){
        var foo = require('foo'),
            bar = require('foo/bar');

        exports.doSomething = function(){
            console.log(foo + bar);
        };
    });

A module is only treated as a CJS module if it doesn't contain a dependency
array and the definition function contain at least one parameter.

More information:

    http://requirejs.org/docs/api.html#cjsmodule
    https://github.com/jrburke/requirejs/wiki/Differences-between-the-simplified-CommonJS-wrapper-and-standard-AMD-define

Member

joewalker commented Jun 17, 2013

Pulled to my branch - will merge to mozilla later.

joewalker closed this

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