Less compile errors not communicated effectively

[jimrhoskins]

When parse error occurs, silently sends a blank file

[benfoxall]

I hacked in something to display parse errors by inserting content with the :before pseudo selector.

if(err){
  css = "body:before{content:'lesscss: "+err.message+"';margin:1em;display:block;color:red;}" + css;
}

(Just before responding with the rendered css in connect-lesscss.js)

Is this an approach that you'd be interested in?

[jimrhoskins]

Yeah, that's one of the solutions I was thinking of. Got sidetracked. I was originally thinking to log the error to stderr at the minimum, but applying it to the page is a great idea.

Any chance you could submit a pull request?

[benfoxall]

sure!

I've been trying to run the tests, though I get an error Error: Cannot find module 'test/support/http' - which seems to be a requirement given in mocha.opts - am I missing a file or something?

[shinmei]

Hi,

I Forked connect-lesscss and made some minor modifications for my personal use.
I submit you a pull request because it's not written in CoffeeScript but in JavaScript.

You may want to check it here : shinmei@9ec9300

It solve the following issues I've had so far :

• err returned by less is lost ;
• fail silently sending back an empty css file ;
• crash when file is not found.