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

Improve error messages to include action name #55

Open
cmbrandenburg opened this issue Jun 4, 2016 · 1 comment
Open

Improve error messages to include action name #55

cmbrandenburg opened this issue Jun 4, 2016 · 1 comment
Assignees
Milestone

Comments

@cmbrandenburg
Copy link
Collaborator

Currently (as of v0.2.0), when an action fails, the resulting error message does not include the action name. Consequently, if the application runs many actions and one fails, then it's not immediately clear which action failed.

@cmbrandenburg cmbrandenburg added this to the v0.2.1 milestone Jun 4, 2016
@cmbrandenburg cmbrandenburg self-assigned this Jun 4, 2016
@cmbrandenburg cmbrandenburg modified the milestones: v0.3.0, v0.2.1 Jun 13, 2016
@cmbrandenburg
Copy link
Collaborator Author

As part of resolving this issue, Chill's Error enum should become an opaque type and applications should instead use a new ErrorKind enum for distinguishing between different CouchDB errors such as, e.g., “not found” vs “document conflict.” This change would free the Error type to become a struct that would contain common fields for all error kinds, including something like an Option<&'static str> for describing the action that failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant