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

unfriendly build.rs errors for stylus #293

Closed
budziq opened this issue May 21, 2017 · 1 comment
Closed

unfriendly build.rs errors for stylus #293

budziq opened this issue May 21, 2017 · 1 comment
Labels
A-Infrastructure Area: CI, Releases C-enhancement Category: Enhancement or feature request

Comments

@budziq
Copy link
Contributor

budziq commented May 21, 2017

Hi,

As mentioned on #292
the build.rs output when of node / npm / nib / stylus is missing are pretty cryptic.

Would you be interested in a PR to make it more friendly?

Moreover, I guess that pulling error-chain as a dependency is out of the question?

@azerupi
Copy link
Contributor

azerupi commented May 21, 2017

Would you be interested in a PR to make it more friendly?

Of course! I will never say no to a PR that improves error messages. 👍

Moreover, I guess that pulling error-chain as a dependency is out of the question?

It is not, I have actually considered using it for some time but never had the time to take a closer look at it. I've heard many people praise its usefulness and simplicity so I am willing to try it out. If you want to pull it in as a dependency I am fine with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Infrastructure Area: CI, Releases C-enhancement Category: Enhancement or feature request
Projects
None yet
Development

No branches or pull requests

2 participants