Deraen / sass4clj

SASS compiler for Clj, Lein and Boot, using Libsass Java wrapper
70 stars 16 forks source link

Add extension option so the output files can end in .css if needed #27

Closed orestis closed 5 years ago

orestis commented 5 years ago

Hi, would you be interested in pulling this in? Shadow-cljs expects the CSS files to end in .css for it to do auto-reloading, and in any case having .css files end in .scss was very confusing for me in the beginning. Thanks!

Deraen commented 5 years ago

Hmm, the extension should already be correct. I guess you tried the SNAPSNOT version?

Boot task replaces the extension: https://github.com/Deraen/sass4clj/blob/master/src/deraen/boot_sass.clj#L86 so I guess it is possible that I missed relevant piece when I build the api ns.

Deraen commented 5 years ago

Fixed by https://github.com/Deraen/sass4clj/commit/91bec0705254a990b1542114d25b50b9137d7f63 & new SNAPSHOT deployed

Deraen commented 5 years ago

Cause was that I copied parts of code from less4clj which has a bit different API and the output extension is set in different part.

orestis commented 5 years ago

Thanks! Yes, I was using the latest master for the api namespace.