search

assemble / assemble-less

Grunt task for compiling LESS to CSS. This task does for less what Assemble does for HTML, making it much easier to modularize and reduce repetition in stylesheets.
http://github.com/assemble/assemble/
MIT License
66 stars 20 forks source link

"process" option doesn't do anything #7

Closed dirkjanm closed 11 years ago

dirkjanm commented 11 years ago

Currently the process option does not do anything. The only lines I found which had something to do with processing are set after a return statement thus they never get executed.

jonschlinkert commented 11 years ago

Thanks for pointing that out. I will remove from docs, that feature has been removed.

dirkjanm commented 11 years ago

Why that? I kinda liked it. Any alternatives?

jonschlinkert commented 11 years ago

Me too actually, it's being "re-introduced" as soon as @doowb refactors part of the engine so that it's easier to add template engines.

dirkjanm commented 11 years ago

All right, I "fixed" it here by moving/changing

if (options.process) {
     lessCode = grunt.template.process(lessCode, options.process);
}

To the concatRender function (before the compiling).

jonschlinkert commented 11 years ago

You know what, I had a brain lapse. When I answered this I wasn't thinking - I pushed up like 4 repos in the last couple of days so I got mixed up..

I pushed up the process option yesterday (I think) but I had to make some changes. I'll be pushing some updates later today. thanks