Skip to content

Conversation

@flaneur2020
Copy link
Member

it seems that errors.Wrap was introduced before fmt.Errorf() in the stdlib was introduced.

the current error message is like:

context canceled: DoRequestFailed

the convention is becoming to put the root cause part into the right side, for example:

do request failed: context canceled

@forsaken628
Copy link
Contributor

github.com/pkg/errors provides stack information that will make debugging easier, otherwise we need to carefully handle the error message

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants