We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Describe the solution you'd like For repos not migrated to OTEL, this linter should also work with the older opencensus spans.
I think it's easy, adding another possible package here (that should be enabled by default):
go-spancheck/spancheck.go
Line 203 in 54ed04d
But RecordError does not exist for this type of spans, so we'll want to ignore those in all checks
RecordError
End() and SetStatus() should be equivalent to otel. RecordError does not exist so should be ignored.
End()
SetStatus()
Additional context
https://pkg.go.dev/go.opencensus.io/trace#Span
The text was updated successfully, but these errors were encountered:
done on v0.5.0
Sorry, something went wrong.
jjti
No branches or pull requests
Describe the solution you'd like
For repos not migrated to OTEL, this linter should also work with the older opencensus spans.
I think it's easy, adding another possible package here (that should be enabled by default):
go-spancheck/spancheck.go
Line 203 in 54ed04d
But
RecordError
does not exist for this type of spans, so we'll want to ignore those in all checksEnd()
andSetStatus()
should be equivalent to otel.RecordError
does not exist so should be ignored.Additional context
https://pkg.go.dev/go.opencensus.io/trace#Span
The text was updated successfully, but these errors were encountered: