Skip to content

apd: drop github.com/pkg/errors#124

Merged
nvb merged 2 commits into
cockroachdb:masterfrom
nvb:nvanbenschoten/remErrors
Sep 9, 2022
Merged

apd: drop github.com/pkg/errors#124
nvb merged 2 commits into
cockroachdb:masterfrom
nvb:nvanbenschoten/remErrors

Conversation

@nvb

@nvb nvb commented Sep 9, 2022

Copy link
Copy Markdown
Contributor

Fixes #123.

Now that the stdlib's "errors" package supports wrapping, use it to eliminate a dependency on "github.com/pkg/errors".

Picks up the "lib/pq" test dependency. Bump go.mod to go 1.17 to allow users of the library to prune this dependency (https://go.dev/ref/mod#graph-pruning).

@mvdan do you mind giving this a look?

nvb added 2 commits September 9, 2022 14:30
Pass `Decimal` by value to `DB.Exec`.

Also use new build tag directive format.
Fixes cockroachdb#123.

Now that the stdlib's "errors" package supports wrapping, use it to
eliminate a dependency on "github.com/pkg/errors".

Picks up the "lib/pq" test dependency. Bump `go.mod` to `go 1.17` to
allow users of the library to prune this dependency
(https://go.dev/ref/mod#graph-pruning).
@nvb nvb merged commit e2030eb into cockroachdb:master Sep 9, 2022
@nvb nvb deleted the nvanbenschoten/remErrors branch September 9, 2022 20:07
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.

please drop github.com/pkg/errors

2 participants