-
Notifications
You must be signed in to change notification settings - Fork 322
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
Rename EndpointResult to Result #235
Comments
PR #258 moves |
@yoshuawuyts I think this can be closed as it has been renamed in 5a2bbd5? |
I've been thinking of ways to improve the error handling story in tide, related to #138, and one of the things that came up is that
tide::EndpointResult
is quite inconvenient to type.I'd like to propose we rename this to
tide::Result
instead, and document that it's intended for use in endpoints. We can then provide examples of theInto
conversions on theResult
type too, which should help with documenting what valid responses are.Proposed
Current
The text was updated successfully, but these errors were encountered: