documentcloud / jammit

Industrial Strength Asset Packaging for Rails
http://documentcloud.github.com/jammit/
MIT License
1.16k stars 197 forks source link

check syntax or report errors #186

Closed foreverman closed 13 years ago

foreverman commented 13 years ago

There is following code in one of our javascript files: SizeUnits = { byte: {u: 'B', U: 'byte'} }; Then some empty asset files(which include the above javascript file) will be generated. Quoting the 'byte' will fix the issue: SizeUnits = { 'byte': {u: 'B', U: 'byte'} };

In my opinion, Jammit should report some errors while failing to generate the asset files instead of generating empty asset files. By the way, I am using jammit 0.4.4.

jashkenas commented 13 years ago

This is a bug in the ruby-yui-compressor gem, which is failing to throw an exception. Feel free to report it over there.