Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show context when printing warnings #202

Closed
JanStevens opened this issue Nov 30, 2015 · 1 comment
Closed

Show context when printing warnings #202

JanStevens opened this issue Nov 30, 2015 · 1 comment

Comments

@JanStevens
Copy link

Hello,

When using include tags the warnings don't reference the included file but rather the concated result. For example my main api has just 4 include tags and I have a warning on line 504. I now have no clue where to look.

I've created a PR for the gulp-aglio package that does include the context of the error. I think this should be added in the main Aglio package, an example:

screen shot 2015-11-30 at 10 53 18

I think this could be done easily in the bin file, optionally It would be better if we had a reference to the original source file.

Related to: #96 but this might be a temp solution?

Regards,

@danielgtaylor
Copy link
Owner

Love it! We should definitely add this.

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

No branches or pull requests

2 participants