Aias00 opened a new pull request, #983: URL: https://github.com/apache/dubbo-go-pixiu/pull/983
## Summary Remove the direct import of `github.com/go-errors/errors` from `pkg/filter/network/dubboproxy/manager.go`, unifying all error handling to `github.com/pkg/errors`. ## Problem `manager.go` simultaneously imported two overlapping error packages: ```go "github.com/go-errors/errors" perrors "github.com/pkg/errors" ``` `go-errors/errors` was only used in this single file (3 call sites), while `pkg/errors` is the project-wide standard (76+ uses). Both provide `Errorf` with stack trace support — the overlap is purely historical. ## Changes | File | Change | |------|--------| | `pkg/filter/network/dubboproxy/manager.go` | Removed `go-errors/errors` import; replaced 3× `errors.Errorf()` → `perrors.Errorf()` | | `go.mod` | `go-errors/errors` moved from direct dependency to `// indirect` | ## Notes - `go-errors/errors` remains as an `// indirect` dependency because `nacos-sdk-go/clients/cache` depends on it transitively — outside our control. - Full project build (`go build ./...`) passes with no errors. Co-Authored-By: Claude <[email protected]> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
