coder / coder/envbuilder

Fix mixed use of `fmt.Errorf` and `xerrors.Errorf`

Open
#367 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
300
Forks
64
Avg merge
20m
Merged PRs (30d)
1

Description

We should use unified error handling in the project. For simplicity we should just use `fmt.Errorf`.

---

nit: I see there is a mix of `fmt.Errorf` and `xerrors.Errorf`. Should we unify them?

_Originally posted by @mtojek in https://github.com/coder/envbuilder/pull/359#discussion_r1778192905_

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the Go repository for uses of xerrors.Errorf and fmt.Errorf, then review the surrounding error-handling conventions. Replace the mixed usage with the chosen unified approach and run the existing project tests to confirm error behavior remains correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.