-
Notifications
You must be signed in to change notification settings - Fork 70
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 onTaskError
option to report errors to node consumers
#808
Conversation
@tmeasday I mentioned this the other day. Right now I just thought to push the formatted error we normally send to the CLI output to the user. I'm curious what you think about adding another formatter to make it shorter or passing along the raw error or other info along with the formatted error. |
onTaskError
option to report errors to node consumers
…by-upload-and' into matt/add-on-task-error-option
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.
LGTM but I agree, let's pass the raw error(s) as well as the formatted version.
5d26bae
to
2d12660
Compare
Adds an option that receives an error when options or task fails. Consumes the same fatalError message that would be logged to CLI for a user error.