Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upUse `console.error` for `stderr` port. #137
Conversation
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
xtian
Feb 16, 2015
Contributor
Can someone let me know if this is a desired change? Happy to close if not.
|
Can someone let me know if this is a desired change? Happy to close if not. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
Looks good to me. Thanks for pinging the PR and making this fix! |
pushed a commit
that referenced
this pull request
Feb 16, 2015
evancz
merged commit abcb93e
into
elm:master
Feb 16, 2015
1 check passed
continuous-integration/travis-ci
The Travis CI build passed
Details
xtian
deleted the
xtian:console-error
branch
Feb 16, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
xtian commentedJan 24, 2015
console.errorlogs an error message to the browser console. It's supported in all browsers back to IE8.This code could actually be simplified to only use the
consoleobject since it's supported in node as well: http://nodejs.org/api/console.htmlThe only difference from writing directly to stdout/err is the addition of newlines to every message. I can amend this PR to make that change if that sounds alright.