search

scalableminds / amd-optimize

An AMD (RequireJS) optimizer that's stream-friendly. Made for gulp.
MIT License
163 stars 29 forks source link

Change how configFile is loaded to allow for calls to require() in the file with the config, common for bootstrapping #44

Closed lamflam closed 9 years ago

lamflam commented 9 years ago

I have a setup where I use index.js to do my config and bootstrapping so that I can have only one script tag. Using the configFile option would crash on my call to require( ["main"], function(main) {...}) since require isn't defined as a function when loading the config.

normanrz commented 9 years ago

Hi, thanks for the PR and sorry for the long wait. I just merged it and posted a new version to npm.