sass-embedded has already been adopted by dartsass-rails, haml, jekyll, nanoc, sinatra, tilt. However, slim was left out due to disagreement over what is the interface contract between slim, tilt, and sass-embedded in #895. There are two points I want to clarify for that:
The tilt API has never guarantee what argument it takes, it is effectively a passthrough tilt_engine.new(...) -> SassEngine.new(...).
Dart Sass (sass-embedded) is a huge breaking change, and it has a redesigned API which is completely different from Ruby Sass.
Since the maintainer of this project does not like the idea of feature detection in the previous PR, this PR implement the next best thing possible. It simply removes the unsupported cache option from the default options. - This is what haml does and nobody complains, so I think it should be fine.
This PR is another attempt of #895 and #889.
sass-embedded has already been adopted by dartsass-rails, haml, jekyll, nanoc, sinatra, tilt. However, slim was left out due to disagreement over what is the interface contract between slim, tilt, and sass-embedded in #895. There are two points I want to clarify for that:
tilt_engine.new(...)
->SassEngine.new(...)
.Since the maintainer of this project does not like the idea of feature detection in the previous PR, this PR implement the next best thing possible. It simply removes the unsupported
cache
option from the default options. - This is whathaml
does and nobody complains, so I think it should be fine.