Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Proposal: Error handling with pkg/errors #82

Open
vcabbage opened this issue Aug 24, 2016 · 1 comment
Open

Proposal: Error handling with pkg/errors #82

vcabbage opened this issue Aug 24, 2016 · 1 comment
Labels
ToDD Rebuild Tasks related to the effort to rebuild ToDD the right way

Comments

@vcabbage
Copy link
Member

Related to #6.

Dave Cheney's https://github.com/pkg/errors allows adding context to errors without losing the original error. It additionally records stack traces for debugging.

I think that this would help bring consistency to the error handling in ToDD and greatly improve the ability to debug issues.

More info on reasoning for the package:
Blog Post: http://dave.cheney.net/2016/04/27/dont-just-check-errors-handle-them-gracefully
GopherCon Presentation: https://youtu.be/lsBF58Q-DnY?list=PL2ntRZ1ySWBdliXelGAItjzTMxy2WQh0P

@Mierdin
Copy link
Member

Mierdin commented Sep 30, 2016

This is a good idea to look at across the board. I know we need to ensure all packages pass relevant error information up the stack, or handle locally if possible (not always doing this well)

I am adding this issue as a member of the Alpha release milestone.

@Mierdin Mierdin added this to the Alpha Release milestone Sep 30, 2016
@Mierdin Mierdin removed this from the Alpha Release milestone Dec 7, 2018
@Mierdin Mierdin added the ToDD Rebuild Tasks related to the effort to rebuild ToDD the right way label Dec 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ToDD Rebuild Tasks related to the effort to rebuild ToDD the right way
Projects
None yet
Development

No branches or pull requests

2 participants