User r.js from require.js

jakewins / brew

Maven plugin for compiling coffeescript, haml-js and aggregating AMD-based js projects.

GNU Lesser General Public License v2.1

91 stars 21 forks source link

User r.js from require.js #8

Closed tbosch closed 13 years ago

tbosch commented 13 years ago

Hello, I created a pull request that uses the r.js from require.js, and also added new optimization options to the OptimizationMojo. I needed this for some new features like the namespace attribute.

Is there a reason that the source did not do this?

Tobias

P.S.: I deleted the CommonJsAmdWrapper, because it was not used in any Mojo. The r.js provides a similar wrapper, however that one only runs on node.

tbosch commented 13 years ago

Hi, I forgot a System.out in OptimizerMojo... ;-)

Tobias

jakewins commented 13 years ago

Dude, this looks legitimately awesome, great work. Thanks for updating tests and stuff as well.