pkg / errors

Simple error handling primitives
https://godoc.org/github.com/pkg/errors
BSD 2-Clause "Simplified" License
8.18k stars 692 forks source link

New release? #152

Closed walle closed 5 years ago

walle commented 6 years ago

Testing out vgo I noticed that this package's last release is v0.8.0 from 2016-09-29 03:48. This is the changes that have happened since then https://github.com/pkg/errors/compare/v0.8.0...master.

Does it make sense to do a new release, at least a patch version if the changes are small, to be able to use the latest code with vgo? Without pinning to a commit that is.

ebati commented 6 years ago

Having a new release would be great, since v0.8.0 cannot pass test with go1.11.

./bench_test.go:26:6: toperr declared but not used vet: typecheck failures

This is fixed at tip as toperr is replaced with GlobalE.

Related go1.11 release note:

Since Go 1.10, the go test command runs go vet on the package being tested, to identify problems before running the test. Since vet typechecks the code with go/types before running, tests that do not typecheck will now fail. In particular, tests that contain an unused variable inside a closure compiled with Go 1.10, because the Go compiler incorrectly accepted them (Issue #3059), but will now fail, since go/types correctly reports an "unused variable" error in this case.

stjohnjohnson commented 5 years ago

@davecheney any chance we could get a v0.9.0 release cut?

dcormier commented 5 years ago

It would be nice to get a new release cut so that WithMessagef is available without having to force dep to use master instead of the last release.

davecheney commented 5 years ago

I'm working on it. Sorry I don't have an ETA.

On 26 October 2018 at 05:48, Daniel Cormier notifications@github.com wrote:

It would be nice to get a new release cut so that WithMessagef is available without having to force dep to use master instead of the last release.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pkg/errors/issues/152#issuecomment-433164093, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAcAx33KiTlmwVGYNuzJqFuprAxzk9Gks5uogeagaJpZM4SX_Q5 .

davecheney commented 5 years ago

v0.8.1 has been tagged.