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

Emit warnings to stderr #61

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
2 participants
@jathak
Copy link
Member

commented Jun 10, 2019

Resolves #14.

This also renames the warn function in lib/src/utils.dart to emitWarning to avoid conflicting with the warn function now exposed by Sass.

@jathak jathak requested a review from nex3 Jun 10, 2019

@JS("process.stderr.write")
external _writeToStderr(String text);

class Stderr {

This comment has been minimized.

Copy link
@nex3

nex3 Jun 11, 2019

Contributor

This seems like it should be in node_io. If it's not there yet, can you send them a PR?

This comment has been minimized.

Copy link
@jathak

jathak Jun 13, 2019

Author Member

I took a look at this, but implementing stderr properly there (rather than just what's needed here) is more of a yak shave than I'm up for at the moment, so I'll close this for now in favor of #62 to fix the compile error.

@jathak jathak closed this Jun 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.