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

refactor: Remove all errors.Wrap and update them with fmt.Errorf. #41

Merged
merged 4 commits into from
Nov 16, 2021

Conversation

shahzadlone
Copy link
Member

@shahzadlone shahzadlone commented Nov 16, 2021

This PR changes all errors.Wraps to fmt.Errorfs.

TODO:

In the next PR (Separate Issue) incorporate the error linter to watch that we always wrap our errors, and compare errors properly through out the code base.

@shahzadlone shahzadlone added this to the DefraDB v0.2 milestone Nov 16, 2021
@shahzadlone shahzadlone linked an issue Nov 16, 2021 that may be closed by this pull request
@shahzadlone shahzadlone self-assigned this Nov 16, 2021
Copy link
Member

@jsimnz jsimnz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove old pkg/errors reference with standard lib errors

@jsimnz jsimnz changed the title Remove all errors.Wrap and update them with fmt.Errorf. refactor: Remove all errors.Wrap and update them with fmt.Errorf. Nov 16, 2021
@jsimnz jsimnz self-requested a review November 16, 2021 19:30
@jsimnz jsimnz merged commit 18894ec into develop Nov 16, 2021
@jsimnz jsimnz deleted the lone/refactor/issue-36-convert-all-old-error-wraps branch November 16, 2021 19:36
@jsimnz jsimnz restored the lone/refactor/issue-36-convert-all-old-error-wraps branch November 22, 2021 22:54
@jsimnz jsimnz added the sred/merged SR&ED activity: Merged label Nov 22, 2021
@shahzadlone shahzadlone deleted the lone/refactor/issue-36-convert-all-old-error-wraps branch December 10, 2021 17:24
@orpheuslummis orpheuslummis mentioned this pull request Jan 29, 2022
shahzadlone added a commit to shahzadlone/defradb that referenced this pull request Feb 23, 2024
…sourcenetwork#41)

* Remove all `errors.Wrap` and update them with `fmt.Errorf`.

* Fix the missing error wrap in the formatting.

* Fixing the `pkg/errors` to `errors`

* Fix the errors.Errorf to fmt.Errorf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sred/merged SR&ED activity: Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Small error handling/wrapping refactor
2 participants