Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

Punch could catch LESS parsing errors #94

Open
smola opened this issue Aug 3, 2013 · 3 comments
Open

Punch could catch LESS parsing errors #94

smola opened this issue Aug 3, 2013 · 3 comments

Comments

@smola
Copy link

smola commented Aug 3, 2013

When running punch s it is sometimes annoying that it fully crashes when less parser throws an error. Punch could handle this so that an error page is returned.

@laktek
Copy link
Owner

laktek commented Sep 14, 2013

I guess this is because you're using an old version of the Less npm module. From 1.4.1, Less module passes the error to callback, instead of raising the exception.

Also, try upgrading to latest Punch (0.5.37 or above), which should log Less parsing errors more descriptively.

@smola
Copy link
Author

smola commented Sep 17, 2013

I have upgraded to Punch 0.5.37 and Less 1.4.2 and it still fails silently. The generated CSS file is empty and punch s just returns status 200.

The project where I've got this problem is here: https://github.com/smola/smola.github.io/tree/punch

@bjfish
Copy link

bjfish commented Oct 17, 2013

@laktek I think @smola is describing something like an in browser error page when parsing errors occur versus just logging to the console when template or css errors occur.

For example, here is an error page from the better_errors gem:
image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants