maccman/sprockets-commonjs — pull request #8 by jamesarosen (2012-07-24 17:11:52+00:00)
Title: Some cleanup

1. move `sprockets/engine` to `sprockets/commonjs/engine` so it doesn't conflict with any other Sprockets Engines
1. add `Sprockets::CommonJS.default_namespace=` so clients don't have to monkey-patch to change it
1. make `Sprockets::CommonJS#prepare` and `#evaluate` protected, like they are for `Tilt::Template` (the parent class)
1. extract helper method `Sprockets::CommonJS#common_js_module?`
1. extract the CommonJS module definition JavaScript into a constant