{"issue":{"title":"Fixed AMD boilerplate to work with loaders that scan the factory function for `require` calls","user":{"login":"bryanforbes","id":204106,"url":"https://api.github.com/users/bryanforbes","avatar_url":"https://secure.gravatar.com/avatar/1fabe88cc065f91ac03606f1fce584b5?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"1fabe88cc065f91ac03606f1fce584b5"},"created_at":"2011-12-28T01:28:58Z","milestone":null,"labels":[],"state":"open","updated_at":"2011-12-28T11:44:54Z","id":2668361,"pull_request":{"patch_url":"https://github.com/Gozala/test-commonjs/pull/5.patch","html_url":"https://github.com/Gozala/test-commonjs/pull/5","diff_url":"https://github.com/Gozala/test-commonjs/pull/5.diff"},"closed_at":null,"html_url":"https://github.com/Gozala/test-commonjs/issues/5","body":"AMD loaders like Dojo's scan the string returned by `factory.toString` for `require` calls and append the dependencies to the list of modules to fetch. Because of this, `undefined` in the factory function ends up being set to the first module fetched. It's also best to avoid the `require` scanning since it's inefficient. I've updated the boilerplate to be both AMD and CommonJS compatible.","number":5,"url":"https://api.github.com/repos/Gozala/test-commonjs/issues/5","comments":1,"assignee":null},"comment":{"user":{"login":"Gozala","id":21236,"url":"https://api.github.com/users/Gozala","avatar_url":"https://secure.gravatar.com/avatar/9289db62d5c34afb4f277a5215e05c83?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"9289db62d5c34afb4f277a5215e05c83"},"created_at":"2011-12-28T11:44:54Z","updated_at":"2011-12-28T11:44:54Z","id":3290117,"body":"Thanks @bryanforbes I'll take a look!","url":"https://api.github.com/repos/Gozala/test-commonjs/issues/comments/3290117"},"action":"created"}