Show sass file on the top of backtrace - #34
Conversation
c084854 to
cc0a7b9
Compare
|
@bolandrm How do you think about this? Is there any blocker to merge? I want this patch to find what is wrong easily in development. |
|
It looks good, no blockers. I just need to get some time to review and test it and then I will merge. |
|
Thank you for your comment! I'm looking forward to being reviewed. |
bca406f to
e8a1c7a
Compare
|
Sorry for the delay, I'll merge this in when I do the next update for libsass 3.3.6. |
|
Hey, I did merge this in, but I made some changes so that we're using the filename and line number information for libsass. it's working in my application, but let me know if you have any issues. thanks for the contribution! |
|
Hi. First of all, thank you for merging this PR! I confirmed your changes and changed my application to use sassc-ruby v1.10.0. I tried some invalid scss and it worked fine. I'll report an issue if I find. |
When SyntaxError is raised, I want to see the broken sass file on error page. This is implemented in
Sass::SyntaxErrortoo.Since we can know filename and lineno only from message, I implemented it by parsing it.
Screen shot
before
after