-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add errors with explanation for the removed options warn and pathToMake #75
Add errors with explanation for the removed options warn and pathToMake #75
Conversation
Found the problem will add another commit soon |
index.js
Outdated
@@ -75,8 +75,10 @@ function compilerErrorToString(err, pathToElm) { | |||
default: | |||
return ("Error attempting to run Elm compiler \"" + pathToElm + "\":\n" + err); | |||
} | |||
} else if ((typeof err === "object") && (typeof err.message === "string")) { | |||
return (JSON.stringify(err.message)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need for ()
around JSON.stringify(err.message)
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
index.js
Outdated
} else { | ||
return ("Exception thrown when attempting to run Elm compiler " + JSON.stringify(pathToElm) + ":\n"); | ||
return ("Exception thrown when attempting to run Elm compiler " + JSON.stringify(pathToElm)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
b6c187a
to
46008dd
Compare
Love it, thank you @razzeee! |
Published as 5.0.1. |
Thanks for the quick feedback loop :) |
This should report helpful errors when using the warn or pathToMake option.
I tried to test it, but still get this error: