Improve DSL error reporting

rstacruz / sinatra-assetpack

Package your assets transparently in Sinatra.

http://ricostacruz.com/sinatra-assetpack/

MIT License

542 stars 97 forks source link

Improve DSL error reporting #124

Closed niallsmart closed 11 years ago

niallsmart commented 11 years ago

Improves the error reporting from the DSL

Provides an error message when options[:from] is missing
Raises Errno::ENOTDIR when options[:from] is not found (instead of silently returning)

j15e commented 11 years ago

I think we could log a warning rather than throwing an error which could avoid having a rescue for the default values (I do not really like the serve '/css', :from => 'app/css' rescue Errno::ENOTDIR part of this), but as it is already a nice improvement I'll merge as is. Thanks!