-
Notifications
You must be signed in to change notification settings - Fork 9
remove dependency https://github.com/pkg/errors #23
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
Comments
Shameless plug: you can also switch it to drop-in replacement |
My preference would be to avoid external libraries which are not part of the standard library as much as possible. |
That is hard to do if you want stack traces with errors, for example. But otherwise yes, standard errors package now is pretty useful on its own as well. |
When this library reports a error, you don't clearly see that the reported error is coming from this library. I wonder how other libraries handle it, or whether there is a convention for it. For example, now we get "invalid OSC address", what if it had a keyword like "osc: invalid OSC address". |
The library should probably export sentinel errors so that callers can use |
Handle errors as discussed here:
https://go.dev/blog/go1.13-errors
The text was updated successfully, but these errors were encountered: