Skip to content

Conversation

@r4victor
Copy link
Collaborator

@r4victor r4victor commented Oct 27, 2025

Fixes #3229

This PR:

  • Drops gerrors and tracerr in favor of standard Go error handling (fmt.Errorf/errors.New).
  • Wraps most unwrapped errors with fmt.Errorf.
  • Fixes code that both logs and propagates errors.
  • Other error handling fixes.

The idea of this PR is to make error handling consistent and use standard Go. Most errors are propagated with fmt.Errorf("some action: %w", err) pattern, effectively building explicit call stack. In some places this may be redundant and verbose but it's still conventional Go error handling.

@r4victor r4victor changed the title Issue 3229 go error handling Use standard Go error handling Oct 27, 2025
@r4victor r4victor changed the title Use standard Go error handling Improve Go code error handling Oct 28, 2025
@r4victor r4victor requested a review from un-def October 28, 2025 06:51
@r4victor r4victor merged commit 34d363a into master Oct 28, 2025
28 checks passed
@r4victor r4victor deleted the issue_3229_go_error_handling branch October 28, 2025 10:23
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.

Make Go error handling consistent

3 participants