Fix CI, and update testify#1284
Conversation
|
The changes in this PR seem safe, but are we concerned that this might make it easy for us to break pre-1.18 users in future PRs? |
We don't currently test multiple versions, so we might be breaking any version other than the one we are testing. 1.18 seemed reasonable, since it is 1 behind the latest go release. |
|
OpenCensus is meant to be in maintenance mode, so I would expect the minimum required versions of its dependencies to remain unchanged unless there's a high priority bugfix or security vulnerability. This should apply to the Go version as well. I would support
I would want OpenCensus-Go to keep building and testing against an old version of Go, and relying on the backwards compatibility of the Go language and stdlib for newer versions. Happy to discuss in person if I'm misunderstanding the purpose of this PR. |
|
@punya |
Address #1283 (comment)
Also, update testify, and run
go fmt.Fixes #1282